Difference between revisions of "Developer Meetings/20120904"

From Slicer Wiki
Jump to: navigation, search
(Created page with 'Attendees: == To discuss == == Conclusion ==')
 
 
(17 intermediate revisions by 4 users not shown)
Line 2: Line 2:
  
 
== To discuss ==
 
== To discuss ==
 +
* Feature Freeze and outstanding bugs
 +
** http://na-mic.org/Bug/roadmap_page.php
 +
** [http://na-mic.org/Mantis/view.php?id=1180 1180]: Slicer might load the wrong Python and other dlls.
 +
* Testing of tutorial functionality
 +
** [http://www.na-mic.org/Wiki/index.php/RSNA_2012 NA-MIC @ RSNA]
 +
** [http://www.na-mic.org/Wiki/index.php/RSNA2012_Planning Tutorial and Demo Planning]
 +
** What can we automate?
 +
** What will be done manually and by whom?
 +
* [https://github.com/Slicer/ExtensionsIndex/issues/67 iGyne extension]
 +
* ITK4 dashboards setup: help is needed with the following machines:
 +
** http://slicer.cdash.org/buildSummary.php?buildid=38107
 +
*** Remove [https://github.com/fedorov/Slicer4DashboardScripts/blob/master/Lebesgue_Win7-64bits_slicer_release_nightly.cmake#L59 line 59]
 +
*** Comment [https://github.com/fedorov/Slicer4DashboardScripts/blob/master/Lebesgue_Win7-64bits_slicer_release_nightly.cmake#L109 line 109] and  <code>set(CTEST_GIT_COMMAND "C:/Program Files (x86)/Git/bin/git.exe")</code>. Full path is required !
 +
** http://slicer.cdash.org/buildSummary.php?buildid=38108
 +
*** I would suggest not to build extension for the moment. <code>set(WITH_EXTENSIONS FALSE)</code>
 +
*** I think packaging of Slicer ITKv4 should be done after the following issue is addressed: [http://na-mic.org/Mantis/view.php?id=2115 2115]. Indeed, we need a mechanism to distinguish packages that will be uploaded on Midas.
 +
*** Note also that be enabling <code>WITH_EXTENSIONS</code>, it means that:
 +
**** No extensions will be built with there is an error while building Slicer.
 +
**** It applies only to the "testing" extensions available in <code>Slicer/Extensions</code> folder.
 +
*** Extensions should be built instead using [https://github.com/Slicer/Slicer/blob/master/Extensions/CMake/SlicerExtensionsDashboardScript.TEMPLATE.cmake SlicerExtensionsDashboardScript.TEMPLATE.cmake]
 +
* CTK patch for review and integration
 +
** Could a CTK developer please review this pull request: https://github.com/commontk/CTK/pull/230 ?
 +
*** Done! Thanks, Jc!
 +
** Helps address:  http://na-mic.org/Mantis/view.php?id=2460
 +
*** Now closed.
  
 
== Conclusion ==
 
== Conclusion ==
 +
 +
* Introduced [http://www.kitware.com/company/team/jhaveri.html Sankhesh] (Kitware Inc) to the rest of the team. Sankhesh will also be helping us to improve Slicer.
 +
 +
* Reviewed list of outstanding bug
 +
** Nicole and Steve discussed sceneview related issues
 +
** Issue 1180 has been targeted for 4.3 since it doesn't impact the user
 +
 +
* Discuss how tutorials will be tested
 +
** Agree on dividing the list of tutorials and having a developer being the "owner" and responsible for coordinating the testing of it
 +
** Steve will be contacting each developer with a list of tutorials
 +
** Tutorials should be updated for Slicer4
 +
 +
* Review how Andriy could fix its ITKv4 dashboard scripts
 +
 +
* After iGyne maintainer will have fix the corresponding description file and done some basic testing, extension will be integrated. See [https://github.com/Slicer/ExtensionsIndex/issues/67 here]

Latest revision as of 20:55, 4 September 2012

Home < Developer Meetings < 20120904

Attendees:

To discuss

Conclusion

  • Introduced Sankhesh (Kitware Inc) to the rest of the team. Sankhesh will also be helping us to improve Slicer.
  • Reviewed list of outstanding bug
    • Nicole and Steve discussed sceneview related issues
    • Issue 1180 has been targeted for 4.3 since it doesn't impact the user
  • Discuss how tutorials will be tested
    • Agree on dividing the list of tutorials and having a developer being the "owner" and responsible for coordinating the testing of it
    • Steve will be contacting each developer with a list of tutorials
    • Tutorials should be updated for Slicer4
  • Review how Andriy could fix its ITKv4 dashboard scripts
  • After iGyne maintainer will have fix the corresponding description file and done some basic testing, extension will be integrated. See here