measure_thermal_behavior icon indicating copy to clipboard operation
measure_thermal_behavior copied to clipboard

Using any integer larger than 0 for COOL_DURATION likely results in the script running a whole day

Open Scroey opened this issue 1 year ago • 2 comments

Probably faulty math or wrong moment to evaulate the term.

wtf minutes The error is likely in https://github.com/alchemyEngine/measure_thermal_behavior/blob/0098166fe155e64352bd6d7eef2f6a1a1760725b/measure_thermal_behavior.py#L492

I used HOT_DURATION = 3 and COOL_DURATION = 1 Error occurred both when the end of the measurement would have been just after midnight and in the middle of the day, so it is probably not a calendar-day related issue.

Scroey avatar Apr 22 '24 13:04 Scroey

@Scroey It looks like to me like it's just a problem with calculation of the remaining time to print..

Did it actually run beyond the expected completion time, or did you cancel it?

alchemyEngine avatar Apr 22 '24 19:04 alchemyEngine

With the prospect of it taking a whole day I killed the run both times this was happening half an hour early. I'll keep it running next time. Thank you for investigating.

Scroey avatar Apr 22 '24 20:04 Scroey