-
-
Notifications
You must be signed in to change notification settings - Fork 150
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
Program crashed while export to csv log file #501
Comments
I have tested a 3.5 version and it has worked. I've tested latest master and it has crashed. Maybe some memory problem, your file generates a 244Mb CSV file that we keep in memory to download. I will test if I can find something that makes it less stable. |
I can confirm... my 3.5 version worked because it was a custom 64 bit version build. Export to CSV your file takes more than 3Gb of memory. I optimized a little the code some time ago, but is clear that it needs more work on it. The 64bit of the Blackbox can work with it, the 32bit not and crashes. On the other side, @mikeller maybe is time to publish a x64 version. I doubt there is too much people using 32bit Windows versions now. We don't support Windows XP or Windows 7 anymore... What do you think? |
64bit windows version will be good solution i think. |
Yeah, I think we can switch to publishing the 64 bit version for Windows. |
@hyperion11 I have pushed a PR to migrate to win64. Once merged the nightlies versions of the Blackbox Log Viewer will be 64bit versions. Until then, if you want to test it I think you can download it from here is where Azure has published the artifact, but will only be available for some time. |
For those still using
Output will speak for itself. |
link to log file
https://1drv.ms/u/s!AuFPVI_eDaZbhY1C00z8whcSC4KQIQ?e=BjV6gV
try stable 3.5.0 and latest nightly. Same problem - program crash.
FC Omnibus F4, fw Betaflight 4.2.5
The text was updated successfully, but these errors were encountered: