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

Remove start timer button - trigger timer start when starting annotation #57

Open
laurentletg opened this issue Aug 14, 2024 · 2 comments · May be fixed by #56
Open

Remove start timer button - trigger timer start when starting annotation #57

laurentletg opened this issue Aug 14, 2024 · 2 comments · May be fixed by #56
Labels
good first issue Good for newcomers

Comments

@laurentletg
Copy link
Collaborator

Summary

  • Remove the Start Timer button - re : extra step that is not required.
  • Annotation time recording could start when the user pushes paint or erase, or switch to the segment editor module.

Describe the solution you'd like

  • Call the start timer method from appropriate methods or buttons/signals.
@delphinepilon delphinepilon added the good first issue Good for newcomers label Aug 14, 2024
@AcastaPaloma
Copy link
Collaborator

would it be wise to create a new section called ''configure segmentation'' and include ''configure labels'' within this alongside the options about the timer? it would be better if we present the user with the option as to when the segmentation starts (paint push or segmentation chosen).

@delphinepilon
Copy link
Collaborator

It could be done. The button "Configure labels..." could be renamed to "Configure Segmentation" and the option to display the timer could be included in the label configuration window, above the label table view. If this was the case, the option to include the semi-automatic PHE segmentation tool should also be moved to this window.

However, these changes should be included in another issue. It could even be discussed whether they should be done within the current pull request (#56) for issue #33.

@AcastaPaloma AcastaPaloma linked a pull request Aug 28, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants