-
Notifications
You must be signed in to change notification settings - Fork 1
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
Filter annotations #66
Comments
This feature should be available for any user, as an editor could use it to review only their own annotations, too. |
Large images and videos can have lots of annotations in a single file. Users should be able to review the annotations of a single file, too. |
The filtering could be implemented in a sidebar tab that is similar to the one of the volume overview. We might need "sorting" and "settings" tabs in the future, too. |
While working on this issue, I realized it could be useful to add to the existing API call for image annotations ( |
Vene
Nevermind, I saw that these parts of the API are already implemented in largo. I will extend the existing ones |
Did you have a look at how the filter UI of the volume overview works? I had something similar in mind here because users are already familiar with it. Under the hood the volume overview filter performs one query for each filter rule (i.e. each rule has it's own API endpoint). The results are then combined to the final sequence in the frontend. The alternative would be a single endpoint but with multiple query parameters, as you describe. I don't prefer one of these two as long as the UI is consistent with the volume overview. |
Largo could be used (by an expert) to review the annotations of a single other user. To do this, the annotations need to be filterable by users. The reviewer/expert can select one (or more) user(s) to see only the annotations of the selected user(s) in Largo.
Summary: Filter by
Ideally, the filters could be combined just like in the volume overview and support less than/equals/more than rules (see biigle/core#359).
The text was updated successfully, but these errors were encountered: