Skip to content

[WIP/NeedsFrontend] Annotation merging: allow specifying if segment ids should be remapped#9537

Draft
fm3 wants to merge 1 commit into
masterfrom
merge-volume-remap-optional
Draft

[WIP/NeedsFrontend] Annotation merging: allow specifying if segment ids should be remapped#9537
fm3 wants to merge 1 commit into
masterfrom
merge-volume-remap-optional

Conversation

@fm3
Copy link
Copy Markdown
Member

@fm3 fm3 commented Apr 28, 2026

URL of deployed dev instance (used for testing):

  • https://___.webknossos.xyz

Steps to test:

  • abc

TODOs:

  • Backend
    • Offer the parameter
    • Propagate the paramter to MergedVolume
    • Do not build id maps in this case
  • Frontend
    • Checkbox in UI (only if non-empty volume layers exist?)
    • Always set to false in case of volume layers with fallback segmentation layer?
  • Discussion
    • What should be the default for upload multiple? For import volume data?

Issues:


(Please delete unneeded items, merge only when none are left open)

  • Added changelog entry (create a $PR_NUMBER.md file in unreleased_changes or use ./tools/create-changelog-entry.py)
  • Added migration guide entry if applicable (edit the same file as for the changelog)
  • Updated documentation if applicable
  • Adapted wk-libs python client if relevant API parts change
  • Removed dev-only changes like prints and application.conf edits
  • Considered common edge cases
  • Needs datastore update after deployment

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 28, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 846f1619-c148-4e02-a1c3-f5c6ecb3d8be

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch merge-volume-remap-optional

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@fm3 fm3 changed the title WIP Annotation merging: allow specifying if segment ids should be remapped [WIP/NeedsFrontend] Annotation merging: allow specifying if segment ids should be remapped May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Merging volume annotations: allow specifying if ids should be remapped

1 participant