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]

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:
- 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.