- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Jul 27 2020
So I am not quite sure that about the re-init problem but to further clarify my list:
Jul 24 2020
I would suggest that you land it for now since it solves the bug and I will create another task in which I also take a look into the stepper problem. But both is fine for me :)
Jul 22 2020
Jul 21 2020
Most likely errors:
- segmentation is always performed on timestep 0 (Threshold/ULThreshold)
- dynamic segmentation is converted into a 3D segmentation (Otsu, Fast Marching, Picking)
Jul 20 2020
Just as a remark: I am getting the exact same error, if I want to load a dcm image from a folder that contains more than one image. Those images originate from a cadaver study which means that before acquisiton most of the patient input data is left blank (study,series etc). If I put the files into separate folders, they are loaded correctly.
Jul 11 2020
I totally forgot to claim the task but I also worked on it *sigh*. Your fix in D346 solves the problem, but I wondered if function initialize() should really be called if the activated() has not been called yet. An alternative would be to have an additional flag "m_activated" or likewise.
Jul 9 2020
Substract Tool also crashes, Paint, Wipe and Correction works
If timestep != 0, the m_CurrentTimeStep is changed with SetCurrentTimeStep(t). In this function Initalize() is called without the m_SmoothFilter initialized before. m_SmoothFilter equals nullptr.
Probably a similar to T24767 (sorry for re-opening but maybe the information from that one helps to narrow it down).
I acutally produced the crash even if the timesteps where valid in both images (contrary to the description).
Although the task was closed I can still reproduce the error:
Jul 8 2020
On the develop (neither debug nor release) I also cannot reproduce the bug, so I think we can close it for now.
Jul 3 2020
Plugin name | dynamic timestep =0 | dynamic timestep != 0 | static t=0 | static t =2 |
Threshold | performs segmentation on all timesteps | performs segmentation on all timesteps | performs segmentation on timestep 0 | performs segmentation on timestep 0 |
UL threshold | performs segmentation on all timesteps | performs segmentation on all timesteps | performs segmentation on timestep 0 | performs segmentation on timestep 0 |
Otsu | performs static 3D seg on the selected timestep + it only matters which time point is selected not on which it was created | see dynamic t = 0 | performs segmentation on the timestep selected right before execution | performs segmentation on the timestep selected right before execution |
Fast Marching | it doesn't matter on which timesteps the seeds are selected, they all account for initial image / performs segmentation only on the timestep selected, others are blank | see dynamic t=0 | ||
Watershed | segmentation cannot be produced with US4DCyl / LinearModel / performs static 3D seg on the selected timestep | see dynamic t=0 | performs segmentation on timestep 0 | performs segmentation on timestep 2 / it only matters which time point is selected not on which it was created |
Picking | Creates a 3D segmentation with picked region and deletes all other segmentations (in all timesteps) |
Jun 30 2020
Jun 25 2020
But the 3d segmentation has only 1 timestep with duration = sum of durations of all 4D image timesteps, right?
Jun 19 2020
The error described above results from the function add seed that currently takes 3D images only. It can be solved with using a timeselector to crop the current time step and set a 3D seed.
However, the task is related to the discussion on time points versus time steps and should be revised accordingly.
When the workbench is started from scratch it works.
Jun 16 2020
I used a master that was three weeks old, I will double check on the current develop
Jun 10 2020
- Works for dynamic (4D) segmentations but creates the same segmentation for all timesteps
- Could not be reproduced for static segmentations since the workbench crashes during the creation of a static segmentation (related to T27476)
- Works for timestep 0
- Independent from segmentation tool so might be a problem of the selection concept not the segmentation
Jun 8 2020
@gaoh: Can you tell what you mean by dynamic mask?
I will leave the task open since the handling in the respective class might need further refactoring to fully understand the behavior (as also mentioned by @kalali in the D279 review).
I remove the MITK2020 tag for now since the mentioned bug is resolved by the fix in D279.
Task T26596 is mentioned but does not correlate with the Topic asaik...maybe transposed digits in the task number?
May 26 2020
Apr 23 2020
Apr 22 2020
Apr 15 2020
The reason seems to be that a BoundingShape as such is 3D with an index of size=3 that does not equal the index array of the 4D image. I will test the fix for other images to make sure.
Pushed new branch T27067-ImageCropperCorrect4D.
This was also communicated to Siemens and only occured when loading their early stage clinical prototype data. Can not be reproduced with the newer data. The task can be closed for now.
Mar 27 2020
Mar 12 2020
As stated before, we encountered more examples that raised the unknown exception, so I don’t agree to lower the priority on that.
Mar 3 2020
The latest Release installer