neuropoly / slicer-manual-annotation

MIT License
1 stars 1 forks source link

Make display of timer optional i.e. configurable upon startup #33

Open delphinepilon opened 2 months ago

maxradx commented 2 months ago

Indeed, I would remove any timer display in the module and make segmentation time of each segment (as currently done) automated. Rationale: the user should not see any timer while performing annotation since this can cause Hawthorne effect (bias)

delphinepilon commented 3 weeks ago

Specifications

Note : take another parameter as an example like is_keyboard_shortcuts_requested.

Expected behaviour If the configuration value is False, the timer should not be visible to the user. If the configuration value is True, the timer should be visible to the user. In both cases, the duration is written to the *SegmentationInformation.csv file. It is a useful metric that should be kept.

AcastaPaloma commented 3 weeks ago

should we still use start/pause/restart buttons for the case where the configuration value is False? If not, what would be the cue to start the timer in the background?

delphinepilon commented 3 weeks ago

I like the idea of keeping the start / pause / restart buttons. It gives the user the opportunity to temporarily stop the timer in the middle of an annotation. Otherwise, the duration of the annotation could be less accurate. I don't know what others think?