Project

General

Profile

Actions

Emulator Issues #8316

closed

Disable changing Emulated CPU Clock Override in Netplay

Added by JMC4789 over 9 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
-
Category:
Netplay
% Done:

0%

Operating system:
N/A
Issue type:
Bug
Milestone:
Regression:
No
Relates to usability:
Yes
Relates to performance:
No
Easy:
No
Relates to maintainability:
No
Regression start:
Fixed in:

Description

Over the past 24 hours, I've realized that I shouldn't have trusted users to use this feature correctly.

Actions #1

Updated by JosJuice over 9 years ago

Why? If it's about desyncs happening, the setting could just be synced. Or do we not want users to be able to force a clock override on other users? In that case, the clock override could be disabled unless all users set it to the same value, so that there still is some way to use it if everyone actually wants to.

Actions #2

Updated by JMC4789 over 9 years ago

Sure, but I didn't know if anyone would want to go through that much trouble. That's obviously a good idea, but I just wanted to make sure people couldn't cause desyncs with it.

Actions #3

Updated by rachelbryk over 9 years ago

IMO, it should be synced not disabled.

Actions #4

Updated by JosJuice over 9 years ago

  • Status changed from New to Accepted
  • Milestone set to Current
  • Relates to usability set to Yes
  • Category set to netplay
Actions #5

Updated by JMC4789 over 9 years ago

Here I was assuming people would think it was too much work to sync it properly.

Actions #6

Updated by JosJuice over 9 years ago

I would still consider the issue fixed if it just is disabled. After all, everything is in sync if the setting always is set to the same value :P

Netplay already syncs some settings, right? I don't know exactly what the code is like, but it shouldn't be too hard to add another setting then. The last suggestion I had is more complex, but that's not really necessary.

Actions #7

Updated by JMC4789 over 9 years ago

The problem is that you can change this setting as the game runs. On/Off being synced would be really easy and nice. Even just a synchronization at startup would be better than current. Problem is, you can turn it on during netplay and people don't realize "Hey, this could cause a desync."

Actions #9

Updated by rukariosake over 9 years ago

#7 could be something to do for issue 8315.

Actions #10

Updated by rachelbryk over 9 years ago

Err, it already is synced??

Actions #11

Updated by JMC4789 over 9 years ago

It's already synced apparently. Never noticed.

Actions #12

Updated by JMC4789 over 9 years ago

  • Milestone deleted (Current)

Because it's synced at startup at least, the only time this would be a problem is if users were actually actively trying to mess things up or even more inept than usual.

Actions #13

Updated by JMC4789 over 9 years ago

  • Status changed from Accepted to Fixed

Fixed

Actions

Also available in: Atom PDF