Improve Proofreading: Add clear button for auxiliary skeletons/meshes & provide merge/split via context menu on voxel #6459
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…e meshes and skeletons after proofreading
philippotto
commented
Sep 14, 2022
hotzenklotz
approved these changes
Sep 14, 2022
…#6464) * implement splitting and merging in proofreading via context menu on non-node position * parallelize some async operations * also validate unmapped ids when doing node-less split; refactor; don't center active node when reloading skeleton * also expose min-cut in context menu when no node is clicked * also remove trees when cleaning up that were added due to reloads * clean up a bit * update changelog * keep currently active node despite of reloading the agglomerate skeleton after split/cut * improve wording in connectome tab * remove the split-only option from the context menu (when not rightclicking a node) * update changelog * Update frontend/javascripts/oxalis/model/sagas/proofread_saga.ts Co-authored-by: Tom Herold <[email protected]> * remove unnecessary log * fix data loading route when floating positions are used * only show proofreading tool if an agglomerate mapping is selected; don't show merge/min-cut context menu items if proofreading tool is not even shown * fix ts error * pretty Co-authored-by: Tom Herold <[email protected]>
philippotto
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Also see #6464 which is also contained in this PR.
URL of deployed dev instance (used for testing):
Steps to test:
Issues:
(Please delete unneeded items, merge only when none are left open)