View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003715 | Slicer4 | Module SubjectHierarchy | public | 2014-05-27 14:27 | 2014-07-14 09:18 |
Reporter | jcfr | Assigned To | pinter | ||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Product Version | |||||
Target Version | Slicer 4.4.0 | Fixed in Version | Slicer 4.4.0 | ||
Summary | 0003715: Fix failing test vtkSlicerSubjectHierarchyModuleLogicTest | ||||
Description | Slicer r23245
276: Test command: /home/jchris/Projects/Slicer-2-SuperBuild-Release/Slicer-build/Slicer "--launcher-no-splash" "--launch" "/home/jchris/Projects/Slicer-2-SuperBuild-Release/Slicer-build/bin/qSlicerSubjectHierarchyModuleCxxTests" "vtkSlicerSubjectHierarchyModuleLogicTest" | ||||
Tags | No tags attached. | ||||
@Csaba: In the TestNodeAssociations test, at line 355 [1], if this helps you to resolve the issue, it returns the subject hierarchy node (vtkMRMLSubjectHierarchyNode7) whereas the associated node (vtkMRMLModelHierarchyNode1) is probably expected: 8<----8<----8<----8<----8<----8<----8<----8<----8<----8<---- |
|
@Csaba: This is an attempt to fix the failing test ... that brings me further but now I have issue with hardening transform. See https://github.com/jcfr/Slicer/compare/3715-attempt-to-fix-associated-hierarchy-node-getter |
|
Strange. The test worked for me a few days ago, and nothing changed in the SH MRML or logic since then. |
|
The test is also failing using the branch [1] built against VTKv5 and also failing on the nightly dashboard for the Linux, MacOSX and Windows In case of nested hierarchy, what is suppose to return the function: static vtkMRMLHierarchyNode* GetAssociatedHierarchyNode(...) For example, looking at the example of PopulateScene [2], given the node "model21", should we expect the direct parent (1) "SubjectHierarchyNode" or the grand parent (2) "ModelHierarchyNode" If we expect (2), the hack I am doing in [3] is along the right way ... [1] https://github.com/cpinter/Slicer/tree/subject-hierarchy-integration [3] https://github.com/jcfr/Slicer/commit/d83c46bcede04a80de934ae0d351bc48fefc2481 |
|
Additionally, checking the comment in [1], I can read: Note: This must be used instead of vtkMRMLHierarchyNode::GetAssociatedHierarchyNode, because nested associations have been introduced to avoid conflicts. |
|
It seems that the PopulateScene method is incorrectly creating the nested association: ModelHierarchyNode -- SubjectHierarchyNode -- ModelNode (model22) instead of SubjectHierarchyNode -- ModelHierarchyNode -- ModelNode (model22) |
|
Exactly. The nested association should look like |
|
Arfff ... not sure to fully understand the intricacy of nested association. Would be great if you could have a look. Thanks |
|
In fact, the code is doing the right thing .. the comment in the test was just misleading. |
|
There was an issue with vtkMRMLHierarchyNode::GetAssociatedHierarchyNode, namely that when the UpdateAssociatedToHierarchyMap function was called before getting the associated node, subject hierarchy's GetAssociatedNode was called, not the function from the base class, which caused the nested associations not to work properly. Fixed in rev23370. Also it was discovered that the harden transform function did not work properly, thus this test failed, because it also tested the harden transform feature of subject hierarchy. Fixed by Alex Y. in rev23391 |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2014-05-27 14:27 | jcfr | New Issue | |
2014-05-27 14:27 | jcfr | Status | new => assigned |
2014-05-27 14:27 | jcfr | Assigned To | => pinter |
2014-05-27 14:27 | jcfr | Target Version | => Slicer 4.4.0 |
2014-05-27 19:27 | jcfr | Note Added: 0011966 | |
2014-05-28 00:27 | jcfr | Note Added: 0011967 | |
2014-05-28 05:13 | pinter | Note Added: 0011968 | |
2014-05-28 06:07 | jcfr | Note Added: 0011969 | |
2014-05-28 06:09 | jcfr | Note Edited: 0011969 | |
2014-05-28 06:16 | jcfr | Note Added: 0011971 | |
2014-05-28 06:35 | jcfr | Note Added: 0011972 | |
2014-05-28 06:42 | pinter | Note Added: 0011973 | |
2014-05-28 06:42 | jcfr | Note Added: 0011974 | |
2014-05-28 06:46 | jcfr | Note Added: 0011975 | |
2014-06-30 12:38 | pinter | Note Added: 0012112 | |
2014-06-30 12:38 | pinter | Status | assigned => resolved |
2014-06-30 12:38 | pinter | Fixed in Version | => Slicer 4.4.0 |
2014-06-30 12:38 | pinter | Resolution | open => fixed |
2014-07-14 09:18 | jcfr | Status | resolved => closed |