-
Notifications
You must be signed in to change notification settings - Fork 10
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
Parser errors #2
Comments
Where did you get the demo you're parsing? I'd like to run it and see what's going on. |
I checked on different demos. |
Oh, I see the problem. I'm guessing you installed the 0.0.1 package. I committed updated protobuf files (the cause of the error you're getting) and some bug fixes a few days ago and haven't had time to make a new release. I'm going to try to do that this weekend. If you install from source you'll be able to parse these demos. |
How could I possibly do that ? |
Check out the source and then in the in the top-level directory (with setup.py) run:
|
|
Did you clone this repo and then run the install command from inside that? |
I downloaded zip, unpacked it to desktop folder, then i went to directory and run commands you gave me. |
I tried to make a |
Another bunch of errors
And with another demo
|
Can you give me a link to that demo? |
Well ? |
Is this with the original demo in OP? did you update the protobuf files? Can you try appending this into netmessages.proto and rebuilding?
|
OK, I'll try |
demo files >1.dem >
2.dem >
Code >
|
can you confirm the svc_entitymsg exception is fixed in dem1? Can you also upload the file on google drive? cant seem to access it from my end.
If you don't care about user messages in your parsing, an easy fix would be to use |
I had the same problem that seems to occur with demos from recent match (i tried with several match demo from GOTV demo from hltv). To compile the source (pip install .) i think you need to have a recent version of protoc (protocol compiler from https://github.com/protocolbuffers/protobuf/releases) as it didn't work with the one shipped with ubuntu (version 2.6.1 compared to the recent 3.6.1). |
@kpei Where did you find that protobuf snippet? Is there a newer version of that file somewhere or did you just write what was missing? I've been using this https://github.com/SteamDatabase/Protobufs to track the protobuf files. That snippet isn't there though. @adrz protoc 2.6.1 should work (it's what gets installed in the Travis CI job). If 2.6.1 isn't working you can open another issue and I'll take a look at it. |
It isn't but it seems like the error at line 518 is trying to locate CSVCMsg_EntityMsg but with the class name CSVCMsg_EntityMessage in the protobuf file so i think appending the struct itself would probably solve the issue. It seems like it worked and the exception no longer shows up. |
|
This issue occurs because CS:GO are allowing new methods on the Protobuf handler. I'm going to PR the updated Protobuf, and also a better error handling for those cases for future. Actually, the Key 69 means the WeaponSound |
fixes these errors:
Maybe you should send a PR ? |
Errors like these still exist. |
|
Right now I made a clone of the repository and got these errors. |
Are you trying to work with POV demos? I have never used this to parse a POV demo. I wonder if those are formatted differently. |
Yes, POV |
I found more errors, can you tell me why they occur ?
code:
Perhaps I should create a new issue for this?
Originally posted by @T1MOXA in #1 (comment)
The text was updated successfully, but these errors were encountered: