keenon / AddBiomechanics

A tool to automatically process and share biomechanics data
https://addbiomechanics.org/
Other
24 stars 5 forks source link

Notifying processing. If not finished in 6 hours, ask to reprocess. #249

Closed AlbertoCasasOrtiz closed 3 months ago

AlbertoCasasOrtiz commented 3 months ago

If the processing flag or the slurm flag exist for more than 6 hours, tell the user to reprocess.

@nickbianco I am not sure how should we procced with the "Waiting for server" status (#248), since there are no flags or files that I know of that we can check for times. In the code, it is in "Waiting for server" status only if there are no other flags, like the processing, queued, error or results flags present.

Should we create a new flag for that case?

nickbianco commented 3 months ago

Should we create a new flag for that case?

Maybe we should, and if it's been in the "Waiting for server" state for a long time (e.g., 24 hours) we then ask to reprocess?

AlbertoCasasOrtiz commented 3 months ago

if it's been in the "Waiting for server" state for a long time (e.g., 24 hours) we then ask to reprocess?

Yes, that would work and would give us more control over states. In the current version, it asks for reprocess only if it has been in the "processing" or "queued" status for 6 hours (I choose randomly). I think we could add the "waiting for server" status to it and change it for 24 hours.

AlbertoCasasOrtiz commented 3 months ago

I changed the time to 24 hours as we discussed, and I found a solution for the Waiting for Server" status.