When the segmentation has multiple groups, labels, instances before saving, closing the project after saving it will crash
| rMITK MITK | |||
| Restricted Differential Revision | rMITKd951ff141fda Correctly handle segmentation observers in interpolation | ||
| Restricted Differential Revision | rMITK51d604bd8dfc remove observers when working data is changed or deleted | ||
| Restricted Differential Revision | rMITK0a303fed6cd3 Fix segmentation serialization by changing serializer name back | ||
| Restricted Differential Revision | rMITK074e26fd7843 rename MultiLabelSegmentationSerializer back to LabelSetImageSerializer | ||
Crash seems to occur because the SurfaceInterpolationController observer logic is broken. It seems like registered observers ar never removed only added.
E.g. also if the segmentation is set to nullptr, nothing is removed from the old segmentation. That leads to the crash, because the events are triggered by the old segmentation when session is closed, but the controller has only a null ptr at this point.
The crash should go away if the observers are managed/removed correctly.
Serialization error: I already renamed LabelSetImageSerializer into MultiLabelSegmentationSerializer, but the data class still names LabeSetImage. I forgot that our scene serializer logic deduces the serializer by <className>+Serializer.
Thus it breaks now. I guess the simpelest solution for now would be to rename the serializer class back.