Difference between revisions of "Developer Meetings/20170606"

From Slicer Wiki
Jump to: navigation, search
Line 23: Line 23:
 
** Slicer/SlicerExecutionModel-specfication makes sense
 
** Slicer/SlicerExecutionModel-specfication makes sense
 
** In future, could also have a  yaml/json version of the spec
 
** In future, could also have a  yaml/json version of the spec
 +
 +
* SEM - If method are not used in extension, we could:
 +
** rename SetParameterDefaultValue/GetParameterDefaultValue to SetParameterValue/GetParameterValue
 +
** add "correct" implementation
  
 
* EMSegment to git:
 
* EMSegment to git:
Line 28: Line 32:
 
** to speed up download, could also use shallow clone ? Look like specifying a commit will not work, a branch name need to be used
 
** to speed up download, could also use shallow clone ? Look like specifying a commit will not work, a branch name need to be used
  
* DICOM https://github.com/pieper/Slicer/tree/dcmtk-for-scalar-volume
+
* DICOM: Use of DCMTK to reach Dicom: https://github.com/pieper/Slicer/tree/dcmtk-for-scalar-volume

Revision as of 14:38, 6 June 2017

Home < Developer Meetings < 20170606


Update

  • Update of VTK: good progress. Talked with David Gobbi to indentify the best strategy to add support for Hierarchy file now required for wrapping
  • EMSegment:

To Discuss

  • SlicerExecutionModel specification home
    • GithubPage associated with SlicerExecutionModel repository (or new repo named SlicerExecutionModel-specification)
    • Add support for "executables" tag and addition to existing "executable" tage: executable path="Foo" type="cpp|python"
  • CLI parameters are accessible in Slicer thanks to function named SetParameterDefaultValue/GetParameterDefaultValue (doc / parameter parsing): possible improvement of the documentation or a more explicit name?
  • Adding DICOM application settings panel.

Conclusion

  • Home for SEM spec:
    • Slicer/SlicerExecutionModel-specfication makes sense
    • In future, could also have a yaml/json version of the spec
  • SEM - If method are not used in extension, we could:
    • rename SetParameterDefaultValue/GetParameterDefaultValue to SetParameterValue/GetParameterValue
    • add "correct" implementation
  • EMSegment to git:
    • Makes sense.
    • to speed up download, could also use shallow clone ? Look like specifying a commit will not work, a branch name need to be used