Difference between revisions of "Developer Meetings/20120904"
From Slicer Wiki
(Created page with 'Attendees: == To discuss == == Conclusion ==') |
m (→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 < 20120904Attendees:
To discuss
- Feature Freeze and outstanding bugs
- http://na-mic.org/Bug/roadmap_page.php
- 1180: Slicer might load the wrong Python and other dlls.
- Testing of tutorial functionality
- NA-MIC @ RSNA
- Tutorial and Demo Planning
- What can we automate?
- What will be done manually and by whom?
- iGyne extension
- ITK4 dashboards setup: help is needed with the following machines:
- http://slicer.cdash.org/buildSummary.php?buildid=38107
- http://slicer.cdash.org/buildSummary.php?buildid=38108
- I would suggest not to build extension for the moment.
set(WITH_EXTENSIONS FALSE) - I think packaging of Slicer ITKv4 should be done after the following issue is addressed: 2115. Indeed, we need a mechanism to distinguish packages that will be uploaded on Midas.
- Note also that be enabling
WITH_EXTENSIONS, 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
Slicer/Extensionsfolder.
- Extensions should be built instead using SlicerExtensionsDashboardScript.TEMPLATE.cmake
- I would suggest not to build extension for the moment.
- 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.
- Could a CTK developer please review this pull request: https://github.com/commontk/CTK/pull/230 ?
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
