CERN / TIGRE

TIGRE: Tomographic Iterative GPU-based Reconstruction Toolbox
BSD 3-Clause "New" or "Revised" License
573 stars 188 forks source link

added human readable time in HH:MM:SS.ms #292

Closed Ayushsunny closed 3 years ago

Ayushsunny commented 3 years ago

improved timing printing in python in HH:MM:SS.ms issue code - #284

Ayushsunny commented 3 years ago

This code does not run as it has a simple yet mayor python error. Please do not submit untested PRs to TIGRE.

I've tested it and had attached screenshot too if you rem? in last PR?

AnderBiguri commented 3 years ago

That screenshot must be from some other code, as I have literally tested this and has an obvious problem that is intrinsic to python basics. Please test it again to see it.

Ayushsunny commented 3 years ago

That screenshot must be from some other code, as I have literally tested this and has an obvious problem that is intrinsic to python basics. Please test it again to see it.

No the screenshot was from this code only. So what error you're getting ?

AnderBiguri commented 3 years ago

Please, test the code to see it yourself, it takes you a second to do so.

This is already taking me x5 times the time it would take me to add this code myself, be respectful of that please.