You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When doing a volume annotation with a fallback layer, saving it, downloading, and then re-uploading it; the option to "load precomputed meshes" is allowed for modified segments but results in the following error:
When selecting "load precomputed mesh" for a not-modified segment, this works as expected.
Expected Behavior
Either the option should not be selectable or loading the precomputed mesh should work without resulting in an error message.
Current Behavior
As described above: In the re-uploaded annotation, loading precomputed meshes is allowed and fails for modified segments. Moreover, the error message reads like the segment that is looked up is 0 which is incorrect.
Steps to Reproduce the bug:
Create a volume / hybrid annotation of a dataset with a fallback layer and precomputed meshes.
Annotate some segment X.
Save the tracing and download the annotation.
Go to the explorative annotations table and reupload the annotation.
In the re-uploaded annotation load a precomputed mesh for the annotated segment X. This results in an error. Loading precomputed meshes for not-annotated segments still works.
The text was updated successfully, but these errors were encountered:
fm3
changed the title
Fix precomputed meshes loading of reuploaded annotation
Loading precomputed meshes is allowed after brushing, but fails
May 27, 2024
I investigated a bit and it turns out the reupload has nothing to do with it.
It is just generally allowed to load precomputed meshes after brushing. It will of course show outdated data for painted-over segments, and it will fail completely for new segment ids created by the brushing user.
I’m not sure what’s the best behavior here and how to build it. One option might be to disallow precomputed meshes completely in this case. Or add a warning. Or figure out which segments have been touched. I think it’s non-trivial to even figure out if brushing has occurred at all (this is why we also added the mappingIsLocked bool, which could possibly in theory be abused for this)
I also think that a warning would be appropriate. We could even show a small "alert" icon next to the "load precomputed mesh" entry when we know that brushing has occurred.
this is why we also added the mappingIsLocked bool, which could possibly in theory be abused for this
yes, there is a 1:1 relation between "a brush operation (or similar) has happened" and "the mapping is locked", right? seems reasonable to use this then. but hang on... I think, after proofreading the mapping is also locked, but precomputed meshes will work fine. so, the logic would need to be isMappingLocked && !mappingIsEditable to find out whether to show the warning.
Context
When doing a volume annotation with a fallback layer, saving it, downloading, and then re-uploading it; the option to "load precomputed meshes" is allowed for modified segments but results in the following error:
When selecting "load precomputed mesh" for a not-modified segment, this works as expected.
Expected Behavior
Either the option should not be selectable or loading the precomputed mesh should work without resulting in an error message.
Current Behavior
As described above: In the re-uploaded annotation, loading precomputed meshes is allowed and fails for modified segments. Moreover, the error message reads like the segment that is looked up is
0
which is incorrect.Steps to Reproduce the bug:
The text was updated successfully, but these errors were encountered: