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

Improve measurement of spent time for tasks/annotations #7543

Closed
philippotto opened this issue Jan 11, 2024 · 0 comments · Fixed by #7627
Closed

Improve measurement of spent time for tasks/annotations #7543

philippotto opened this issue Jan 11, 2024 · 0 comments · Fixed by #7627
Assignees

Comments

@philippotto
Copy link
Member

philippotto commented Jan 11, 2024

Time spent by annotators is measured automatically by wk. TimeSpans are stored in postgres describing the work.

When a user sends an annotation update request to the Tracingstore, it sends a summary on to the webknossos server (remoteWebKnossosClient.reportTracingUpdates)/ WKRemoteTracingStoreController.handleTracingUpdateReport).

Some users have asked that move-only updates, where no segments/nodes were modified should not be counted for the time tracking. The report already contains a significantChangesCount.

An application.conf parameter should be added to disable timetracking for updates with significantChangesCount==0.

Note for testing this: currently, only timespans of tasks are available in the UI. You can still use postgres to inspect stored timespans for explorative annotations. Compare #7524

More sophisticated ideas (that we decided against for the moment) include

  • Setting per project instead of instance-wide application.conf setting
  • For each update, use only the timestamps that stem from “significant changes”
  • Also include timestamps for move-only events except if move-only occurs for more than x seconds

Context

Internal discussion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants