Skip to content
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

Crash on parsing the time if the load is running for more than 24 hours #17

Open
SY-Bisou opened this issue Feb 20, 2022 · 5 comments
Open

Comments

@SY-Bisou
Copy link

Hi Misdoro, i run PX100 since 1/2 year. Till now, it´s open shortly and close the program with the appended message.
Any idea ?
2022-02-12_12h40_30

@misdoro
Copy link
Owner

misdoro commented Feb 20, 2022

🤔 aha, it must be an issue if the load was been running for more than 24 hours...

@SY-Bisou can you try resetting the totals using the physical button on the load?

@misdoro misdoro changed the title PX100 is not starting till windows update Crash on parsing the time if the load is running for more than 24 hours Feb 20, 2022
@misdoro misdoro mentioned this issue Feb 20, 2022
@misdoro
Copy link
Owner

misdoro commented Feb 20, 2022

It will be tricky to fix since the native python time is limited to 23:59:59 :facepalm:

@misdoro
Copy link
Owner

misdoro commented Feb 21, 2022

And the same crash occurs if the timer is set to more than 24h

@SY-Bisou
Copy link
Author

thanks, i try to reset the timer

@misdoro misdoro reopened this Feb 21, 2022
@mbastida123
Copy link

Hi, I have the same problem.

My discharge test takes longer than 24h so it crashes.

One solution would be to used datetime.datetime insted of datetime.time. This would not be the best solution since we are not storing a date but a duration. Or we could simply add a %24 to this line:

Would this work?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants