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
ladder game not reported right #255
Comments
|
Grothe the log says you got disconnected from the server twice, after you died:
|
well, the point is, I didn't die, I won that game, with an ACU close to healthy as can be. But the server result indicates a draw. (Replay shows both lost and the ladder-points fit a draw) |
I meant before he died. |
apoligy accepted ;-) |
Yes, at least in some cases; if both players disconnect, the game will be marked as "ended", but as long as at least one player is connected, #237 will restore the players "game state" once he reconnects. This is necessary in order for the player to report results, I think. #237 will be deployed with the next server update and requires client-side support. (FAForever/client#522) |
Game: 5537057
Scores reported by JaggedAppliance:
Scores Reported by Tokyto_:
Scores reported by Blodir:
Scores reported by AchievedJaguar8:
Related error message:
Then there is also:
Also I found this: Line 367 in 58a9135
|
Hmm, seems like the check Here's an example how it's handled in _process_pending_army_stats: Dynamic-typed languages ftw ;) |
is there maybe a copy paste error? -> Scores reported by AchievedJaguar8 are Blodir. |
@Crotalus I think this is a follow-up error. I looked at the code a few times already, and IMO army_result[1] should always be set at this point.
And yes, @GrotheFAF that was an error, I removed it, thx. |
In the current code
(army 1, 2 reported victory 10 points for that player) and then stats processor check |
there is still issue with rating it seems |
Old issue. Rating code has been rewritten |
ladder game #5497983 is shown and counted like both players lost, but I actually won.
(the server stored replay look right, but shows no score at the end.)
forever.zip
The text was updated successfully, but these errors were encountered: