Developer Meetings/20121009

Attendees:

To discuss

• Tutorial testing status
  • ChangeTracker standing issues:
    • Crash if close slicer w/o closing scene: http://www.na-mic.org/Bug/view.php?id=2590
    • 1 matrix and 12 vtkCommands are leaking: http://www.na-mic.org/Bug/view.php?id=2588
• Discuss commit r21147
  • related issue 2534
• Style and Coding Conventions discussion topics
  • Commit message style. See here
  • Backwards compatibility and deprecation
  • Q_ASSERT vs qDebug, and vtkErrorMacro
• Memory management from 'Create' methods [1]

Conclusion

• Talked about the factory and Linux VM before every body joined
• Initiated discussion about Tutorial testing
  • Nicole mentioned that the Brain Atlas dataset taking more than one hour to load on a linux debug build (SPL workstation)
  • Andriy discussed the crash happening when closing slicer without "closing the scene" first after running ChangeTracker. See issue 2590
  • Andriy also mentioned the work done related the memory leaks associated with ChangeTracker tutorial. See 2588. Julien pointed the Slicer wiki page listing debugging approach that could be used to "Track a crash while accessing already deleted object pointer". See http://www.slicer.org/slicerWiki/index.php/Documentation/Nightly/Developers/Tutorials/Troubleshooting
• Discussed warnings occurring when loading Brain atlas scene (nac-brain-atlas-1.0/brain-atlas-hierarchy+snapshots.mrml):
  • vtkMRMLParser (0x6b3a990): Failed to CreateNodeByClass: vtkMRMLTGParametersNode.
  • Command line module Model Maker is version "4.1" but parameter set from MRML file is version "3.2". Parameter set may not load properly
  • There is a danger here. What if AddNode or RemoveNode were called prior, the NodeIDsMTime would be in sync without having the map in sync.