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

game: added medic statistics to scores #2232

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

MrWaggel
Copy link

Keep track of revives and picked up medkits in the /scores dump.

console_scores_norating
scores_statson

  • RG Revives Given
  • RR Revives Received
  • THG Total Health Given (by medkits and revives)
  • THR Total Health Received
  • EHG Enemy Health Given (enemies picked up medkits)
  • EHR Enemy Health Received

Tested it with bots, seemed to work.

@x0rnn
Copy link
Contributor

x0rnn commented Feb 27, 2023

Can you add it to the weaponstats/accstats too? By default Alt or Ctrl key. Would be nice to also count ammo packs then.

@Aranud
Copy link
Contributor

Aranud commented Feb 28, 2023

refs #1842

@jackeri
Copy link
Member

jackeri commented Mar 12, 2023

Doesn't this just add somewhat of a bloat to the dump? I mean, if we add medic specific stats to it, why not fops specific, or covi?

@HeDo88TH
Copy link
Contributor

Doesn't this just add somewhat of a bloat to the dump? I mean, if we add medic specific stats to it, why not fops specific, or covi?

This is a valid point. I think medic is the most played class so he went for it. Anyways if we go down this route it would be interestng to see the ammo packs given / received. For covert you could show the number of stolen uniforms. And so on...
I think this would be interesting but we have a decision to make before merging this.

@x0rnn
Copy link
Contributor

x0rnn commented Feb 4, 2024

Maybe just Revives Given/Received would make sense then.

Copy link

sonarcloud bot commented Feb 13, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

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

Successfully merging this pull request may close these issues.

None yet

5 participants