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

noElo modoption does nothing #1801

Closed
GoogleFrog opened this issue Aug 20, 2017 · 12 comments
Closed

noElo modoption does nothing #1801

GoogleFrog opened this issue Aug 20, 2017 · 12 comments

Comments

@GoogleFrog
Copy link
Contributor

See this game http://zero-k.info/Battles/Detail/466192?ShowWinners=True

@DeinFreund
Copy link
Member

DeinFreund commented Aug 30, 2017

Could somebody check the database whether noelo was actually registered? I'd assume something already failed in the match setup leading to the value not being set.

@Histidine91
Copy link
Member

Database doesn't store noElo state.

Server tries to read noElo state here before deciding whether to apply Elo effects. I am unable to find any indication that the value is actually written in the place the code looks.

@Sgathec
Copy link

Sgathec commented Dec 9, 2017

Many, many months or even a year ago, noelo sometimes produced the intended result. Is it now guaranteed not to function? I haven't had it enabled for a long time.

@Anarchid
Copy link
Member

Is this obsoleted by the post-hoc administration of what counts?

@sprunk
Copy link
Member

sprunk commented Dec 29, 2017

No. Both players and admins have to take extra steps in addition to setting the modoption which are not necessary and many extra failure cases are created:

  • players don't know they have to contact an admin for the modoption to actually work.
  • players know but don't because of bystander.
  • a player takes the responsibility but still fails to (disconnected, forgot...)
  • admins are pinged but nobody tags the battle due to bystander.
  • admins tag the battle without checking whether the battle really was noelo.

@DeinFreund
Copy link
Member

Tags are set automatically, assuming noelo is properly sent to post battle processing.

@sprunk
Copy link
Member

sprunk commented Apr 10, 2018

I did a check and the infra correctly processes noElo.

@sprunk sprunk closed this as completed Apr 10, 2018
@Sgathec
Copy link

Sgathec commented Aug 22, 2018

http://zero-k.info/Battles/Detail/600713
noelo=1 in a casual room, but casual ratings were changed after the battle.

@sprunk sprunk reopened this Aug 22, 2018
@sprunk
Copy link
Member

sprunk commented Aug 22, 2018

ZK (as in the game content itself) issue. ZeroK-RTS/Zero-K#3223

@sprunk sprunk closed this as completed Aug 22, 2018
@GoogleFrog
Copy link
Contributor Author

This is stupid. Read noElo from the modoptions directly upon game start. Springie sets the modoptions and it is written in the replay.

@GoogleFrog GoogleFrog reopened this Jun 4, 2019
@GoogleFrog
Copy link
Contributor Author

@sprunk
Copy link
Member

sprunk commented Oct 9, 2019

d4a1a38

@sprunk sprunk closed this as completed Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants