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

Init stats to zero #814

Merged
merged 1 commit into from
Aug 1, 2022
Merged

Init stats to zero #814

merged 1 commit into from
Aug 1, 2022

Conversation

nickdnk
Copy link
Collaborator

@nickdnk nickdnk commented Aug 1, 2022

Replacement for #794 as that was in forked repo, making rebasing annoying.

Something like this should fix #654.

Main points:

We now init all stats to zero at the beginning of each round for all players this has not happened for yet. This allows ringers etc. to come in as replacements and still have a proper stats file.

Used the LOOP_CLIENTS in more places.

@nickdnk nickdnk added this to the 0.10 milestone Aug 1, 2022
@nickdnk nickdnk self-assigned this Aug 1, 2022
@nickdnk nickdnk requested a review from PhlexPlexico August 1, 2022 14:36
@nickdnk nickdnk linked an issue Aug 1, 2022 that may be closed by this pull request
@nickdnk nickdnk force-pushed the init_stats_to_zero branch from 21cab9d to 75817cc Compare August 1, 2022 14:51
@nickdnk nickdnk changed the title Init stats to zero Init stats to zero and add Cvar to disable KeyValue stats Aug 1, 2022
Use LOOP_CLIENTS helper in more places
Minor refactors in stats.sp
@nickdnk nickdnk force-pushed the init_stats_to_zero branch from 16bd47d to da9c95e Compare August 1, 2022 18:52
@nickdnk nickdnk changed the title Init stats to zero and add Cvar to disable KeyValue stats Init stats to zero Aug 1, 2022
@nickdnk nickdnk merged commit b8d6aff into development Aug 1, 2022
@nickdnk nickdnk deleted the init_stats_to_zero branch August 1, 2022 19:06
nickdnk added a commit that referenced this pull request Aug 20, 2022
Use LOOP_CLIENTS helper in more places
Minor refactors in stats.sp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stats file
1 participant