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

Bulk create annotation values #1069

Merged

Conversation

gregorjerse
Copy link
Contributor

No description provided.

@gregorjerse gregorjerse force-pushed the feature/annotation_bulk_create branch 7 times, most recently from 826e143 to aa93e69 Compare November 15, 2023 15:07
resolwe/flow/views/annotations.py Outdated Show resolved Hide resolved
resolwe/flow/views/annotations.py Outdated Show resolved Hide resolved
resolwe/flow/views/annotations.py Outdated Show resolved Hide resolved
resolwe/flow/views/annotations.py Outdated Show resolved Hide resolved
resolwe/flow/views/annotations.py Outdated Show resolved Hide resolved
@gregorjerse gregorjerse force-pushed the feature/annotation_bulk_create branch 2 times, most recently from a8c1575 to 01689e5 Compare November 21, 2023 06:05
@gregorjerse gregorjerse changed the title WIP Bulk create annotation values Bulk create annotation values Nov 21, 2023
@gregorjerse gregorjerse force-pushed the feature/annotation_bulk_create branch 2 times, most recently from 209673f to a378aea Compare November 21, 2023 06:33
for value in group:
instance = annotation_map[(value["field"], value["entity"])]
to_delete = value["_value"]["value"] is None
if to_delete:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can write the expression inline and avoid using another variable.

@gregorjerse gregorjerse force-pushed the feature/annotation_bulk_create branch from a378aea to 99a4f85 Compare November 21, 2023 07:23
@gregorjerse gregorjerse force-pushed the feature/annotation_bulk_create branch 2 times, most recently from b758c97 to 6bcb50e Compare November 21, 2023 08:27
@gregorjerse gregorjerse force-pushed the feature/annotation_bulk_create branch from 6bcb50e to bab55be Compare November 21, 2023 08:27
@gregorjerse gregorjerse force-pushed the feature/annotation_bulk_create branch from bab55be to 0df2322 Compare November 21, 2023 08:28
@gregorjerse gregorjerse merged commit 0df2322 into genialis:master Nov 21, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants