Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FightSync Improvements - Ping/TPS Integration #37

Closed
andris155 opened this issue Apr 12, 2019 · 4 comments
Closed

FightSync Improvements - Ping/TPS Integration #37

andris155 opened this issue Apr 12, 2019 · 4 comments
Labels
enhancement New feature or request

Comments

@andris155
Copy link

Fightsync flood on chat, and active: false setting not work. Please fix this.

@CaptainObvious0
Copy link
Owner

Please use the proper reporting format.

I’ll need more information as well. Latency can probably play a big role, so that can be added. For now, remove logging from the actions list.

@andris155
Copy link
Author

The ping 40-50, because server location in France, but players from Hungary.

@CaptainObvious0
Copy link
Owner

Enabling/Disabling the check has been fixed with 0d65210

Further improvements will come in the future. Some sort of latency/tps leniency should be integrated, possibly even attempting to match up estimated packet arrival time with the players ping.

@CaptainObvious0 CaptainObvious0 added enhancement New feature or request and removed Fork Issue labels Apr 12, 2019
@CaptainObvious0 CaptainObvious0 changed the title Fightsync flood, and active: false not work FightSync Improvements - Ping/TPS Integration Apr 12, 2019
@CaptainObvious0
Copy link
Owner

Will most likely remove FightSync check and work on other things found here: #51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants