-
Notifications
You must be signed in to change notification settings - Fork 85
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
"2207 bytes left!" on newer replays. #190
Comments
I think the lotv branch is behind the times unfortunately. My branch https://github.com/dsjoerg/sc2reader/tree/upstream is more On Mon, Feb 8, 2016 at 3:36 AM, OMGTallMonster [email protected]
|
Thanks for the heads up. Your branch works! |
I got the similar error when trying your branch and the branches here:
|
@stevemao if you post the replay somewhere I'll give it a try on my branch and see if there's something easy to fix. By the way, what kind of replay was this? (SC2 LotV? Ladder? 1v1?) |
Try this one: |
Hi @stevemao I just uploaded that replay to GGTracker, which indicates that my branch https://github.com/ggtracker/sc2reader/tree/upstream can parse the replay OK. http://ggtracker.com/matches/7166787 |
Hi @dsjoerg I just tried your branch and got the similar error: BTW: In the "creeptracker.py" there is some code that do these kind of things: "
|
OK this is crazy/interesting. I committed a test to the suite (ggtracker@a15a5f5) that demonstrates that sc2reader can parse the replay. And yet when I run |
OK @stevemao @MLLeKander everything is working. In my environment, it turned out to be a Once I fixed things so that I was running the correct version, all replays are parsed successfully. As I mentioned yesterday, I committed a test the demonstrates that sc2reader can parse the replay provided by @stevemao (http://lotv.spawningtool.com/29785/). I also just tried the original replay provided by @MLLeKander (http://lotv.spawningtool.com/10851/) and it also parsed fine for me, and I uploaded it to GGTracker (http://ggtracker.com/matches/6447606) as well. I'm leaving this issue open, because this issue is about GraylinKim/sc2reader, not ggtracker/sc2reader. |
And by the way everyone, please use https://github.com/ggtracker/sc2reader/tree/upstream and not https://github.com/dsjoerg/sc2reader/tree/upstream !! |
I get an error when trying to parse recent replay files, such as this one.
The output of sc2parse using the lotv branch:
If it's relevant, this is the output from the master branch:
The text was updated successfully, but these errors were encountered: