-
Notifications
You must be signed in to change notification settings - Fork 41
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
some more infos/flags in fast header parser and AI compatibility #98
Conversation
Happy to merge updates to the fast parser. I'm not sure about the 42 bytes after the players. I'm currently parsing rated and unrated matches successfully with |
I forgot to mention, |
You are right, it seems to have nothing to do with rated or unrated. But e.g. this record is parseable with the skip, but not without: I had a few more but i have no clue whats the reason behind those bytes. And i can't see any dependence on any game settings. |
FWIW, it's the 42 bytes before it reads the 26 for version 37. You could do a lookahead and optionally skip those bytes if all else fails. |
@happyleavesaoc Still not all savegames can be parsed, we still have two typical error cases:
All these matches cannot be parsed even with the unmodified version, so in my opinion the status is not getting worse. We have been using this version live for two days now and have already parsed thousands of matches. I would continue to work on the open error cases separately and submit this status now for the merge, if you still have suggestions for improvement, please give me feedback. Best regards! |
Great, I will merge and bump the version then. Thanks! |
Hey,
we're just making our way to the fast header parser.
I have added some flags that we need, is it possible to include them upstream?
Attention: I dived into the binary data a bit and noticed that there seems to be a discrepancy in the player data (where I skip 42 bytes), I thought it was due to RoR, but it seems to be more of a difference between rated and non-rated matches, at least with the skip I can only parse non-rated matches and without the skip only rated matches.
So the PR shouldn't be merged like that, maybe you have an idea how to solve the problem?