Faulty weekly reset from someone that doesn't PvP this week will mess with data
Slivo-fr opened this issue ยท 3 comments
When a players login after weekly reset, with a failed blizz weekly update, honorspy will broadcast the previous week values to other players.
Later, when the weekly blizz update is finaly rolled out, if this players hasn't done any PvP, inspecting will remove it from table but other players will sent back the wrong value again and it will eventually get re-added.
Those edge case may affect the natural pool count and could interfere with estimate/bracket if enough honor was made on the previous week.
I can't think of a way to handle this
Here is an idea :
- We check every player even under 15hk
- If a player under 15hk is already in the table (most likely a failed blizz reset) we set his honor to 0 or update his honor data
Doing this, the player will still show in honorspy, but at least it won't stay at last week honor for the whole week.
Any thought ?
Considering the uncommented code from #210
We could also keep a defined delay past the weekly reset before storing players to avoid storing failed resets at all
Let's say 6 hours for example ?