Subject hierarchy problems after beam changes

[cpinter]

After the recent improvements related to RT plans and beams (#120 and #122) these items in subject hierarchy behave in a wrong way.

I loaded the usual test dataset RANDO PROSTATE and got this in subject hierarchy:

Issues:

1. The plan, the beams, and the MLCs are under the root, but they should be in the study.
2. When I click the eye icon to show a beam, the item disappears from SH
3. The dose volume does not show at all
4. The structure set is now somehow also under the root. Needs to be in the study as well

[cpinter]

The above screenshot was from the DICOM module's SH tree. After switching to the Data module, the following changes happened:

1. The dose volume appeared in the right place
2. The beams beacme children of the plan
3. The Plan's POI markup fiducials node appeared under root

[MichaelColonel]

I can comment some elements about MLC:

Two MLC nodes (table for positions, double array for boundary) are created and added to the scene as this->External->GetMRMLScene()->AddNode(mlcNode);.

1. vtkSlicerDicomRtImportExportModuleLogic::vtkInternal::CreateMultiLeafCollimatorDoubleArrayNode for boundary,
2. vtkSlicerDicomRtImportExportModuleLogic::vtkInternal::CreateMultiLeafCollimatorTableNode for positions.

The part of the code which forms hierarchy: MLC table as a child of the beam, Scan Spot Map data as a child of the ion beam, goes right after scene->EndState(vtkMRMLScene::BatchProcessState); statement in vtkSlicerDicomRtImportExportModuleLogic::vtkInternal::LoadRtPlan method.

It doesn't work if i place that part before the EndState command, even when i switched to the Data module (MLC tables will remain under the root).

[cpinter]

Thanks, I'll look at this. It is possible we should remove batch processing, since only a few nodes are involved.

However, I cannot do much as long as SlicerRT build fails, please see #129

[MichaelColonel]

Correction about MLC nodes in #120 and #122.

Position MLC table node in hierarchy as a child of beam node has been added only with initial ion plan support #124.

[cpinter]

Thanks! I remember that you left some of the SH stuff on us, so no rush with this. What really bothers me is the disappearing items. Maybe @Sunderlandkyl could look at this after the conference (Slicer project week starts on Monday), or me during...

[MichaelColonel]

No rush at all. I thought that the mess in DICOM module's hierarchy tree was not a bug but a feature.

[MichaelColonel]

Please also note, that in #124 i have slightly changed beam node naming:
" BeamName [TreatmentType] : ControlPointIndex" (e.g. "LT LAT PROS [TREATMENT] : CP0").

Of course it can be easily disabled, if it unacceptable!

[cpinter]

I removed batch processing, and item creation and parenting work well. Beam visibility as well.

Not sure, but I think this problem is related to the recent changes in the DICOM module, so now there is an active SH tree view during loading, and it updates itself live while the nodes and items from DICOM are being loaded.

[MichaelColonel]

Some minor change in style that doesn't affect beam visualization, compared to the previous version. #136

[cpinter]

In the latest version subject hierarchy appears and functions normally.