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

ENH: Add support for multi-user segmentation #167

Closed
wants to merge 1 commit into from

Conversation

mehrtash
Copy link
Member

@mehrtash mehrtash commented Dec 12, 2019

This will fix #151.

@mehrtash mehrtash changed the title ENH: support access to other users' annotations ENH: Add support for multi-user segmentation Dec 12, 2019
@fedorov
Copy link
Member

fedorov commented Dec 13, 2019

As I was testing this, I selected the first study from the dataset we are working with, then selected to see Felipe's segmentations, and then Fiona's segmentations, and I got this popup:

image

@fedorov
Copy link
Member

fedorov commented Dec 13, 2019

If I hit "cancel", I can see Fiona's segmentations, but if I switch back to Felipe, I then get this:

image

@fedorov
Copy link
Member

fedorov commented Dec 13, 2019

I think it would be better to show segmentations from another user in non-editable view-only mode, and have a button to explicitly duplicate the segmentations. When user requests duplication of segmentations, it should be easy to check if segmentations for the corresponding structures already exist, and if yes - confirm if they should be over-written. Then once segmentations are copied, they can be loaded under the active user ID, and available for editing. Do you think it would be too difficult to implement this logic?

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.

Support multiuser segmentation
2 participants