-
Notifications
You must be signed in to change notification settings - Fork 3
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
Getting error -2 #53
Comments
-2's the general failure code. I'll need to take a closer look. Can you send/post a link to a replay that's causing this error? I've been parsing replays from last week without issue. |
Will do once I get home from work tomorrow
|
As promised, here are my latest two replays (both won't work) https://github.com/lookapanda/hots-parser-replay |
got them, thanks! will take a look this week |
Cool, thanks!
…On Mon 16. Sep 2019 at 23:56, Evan Shimizu ***@***.***> wrote:
got them, thanks! will take a look this week
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#53?email_source=notifications&email_token=AAF4ESEZLBFBTRM3OP2HHQ3QJ76HDA5CNFSM4IW4J2EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD62UM4Q#issuecomment-531973746>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAF4ESBKKC3UNF5DK43JLM3QJ76HDANCNFSM4IW4J2EA>
.
|
hm, I'm unable to reproduce any error when parsing these files on my machine. What platform are you running on? I'm running a Windows machine using Node 10 and the latest release of this code (tag 2.47.3) I have gotten some reports of parsing issues on other platforms in the past involving a dependent module. |
I‘m running Windows but using WSL (Ubuntu 18.04)
…On Thu 19. Sep 2019 at 20:37, Evan Shimizu ***@***.***> wrote:
hm, I'm unable to reproduce any error when parsing these files on my
machine. What platform is this running on? I'm running a Windows machine
using Node 10 and the latest release of this code (tag 2.47.3)
I have gotten some reports of parsing issues on other platforms in the
past involving a dependent module.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#53?email_source=notifications&email_token=AAF4ESBOJ3PP7GIWRRW3UHLQKPBHXA5CNFSM4IW4J2EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7ENWDI#issuecomment-533256973>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAF4ESAFBZ7PJHU6O7BGDIDQKPBHXANCNFSM4IW4J2EA>
.
|
hmm yeah you might be running into some issues with the stormreplay module used in heroprotocol. That module uses a native extension to accelerate the replay extraction, but sometimes runs into problems. You can try to disable the native extension or possibly try downgrading |
hi @ebshimizu , got same error today with some games played yesterday. Tell me if u have the same issue so we can fix together if u need some help |
There have been a few issues with the base heroprotocol repository (Blizzard/heroprotocol#92) and this package had to swap over to a fallback. First try re-installing the hots-parser package, and then let me know if the issue still persists. |
Latest patch replays, getting error -2
The text was updated successfully, but these errors were encountered: