ITK/Release 4/New Code Contribution Process: Difference between revisions
Daviddoria (talk | contribs) m (moved ITK Release 4/New Code Contribution Process to ITK/Release 4/New Code Contribution Process) |
|||
(13 intermediate revisions by 3 users not shown) | |||
Line 22: | Line 22: | ||
## Gerrit provides a Peer-review mechanism for the patch | ## Gerrit provides a Peer-review mechanism for the patch | ||
## Gerrit uses [[ITK_Release_4/Testing_On_Demand/Tutorial|Cdash@home]] to provide automated testing of patches | ## Gerrit uses [[ITK_Release_4/Testing_On_Demand/Tutorial|Cdash@home]] to provide automated testing of patches | ||
# After Gerrit peer-review, [[ITK/Git/Develop#Merge_a_Topic|merge the topic]]. | # After Gerrit peer-review approval, [[ITK/Git/Develop#Merge_a_Topic|merge the topic]]. | ||
== Internal Modules == | == Internal Modules == | ||
Line 103: | Line 103: | ||
* Alternatives | * Alternatives | ||
** Use the CMake ''find_package(ITK)'' command from the external layer (library or application). | ** Use the CMake ''find_package(ITK)'' command from the external layer (library or application). | ||
** Use ITK as an external project in a Superbuild approach | ** Use ITK as an external project in a Superbuild approach. | ||
*** The Superbuild | *** The Superbuild groups the external layer along with ITK into a larger distribution... | ||
===Examples=== | ===Examples=== | ||
Line 111: | Line 111: | ||
** Wrapping for C# | ** Wrapping for C# | ||
== | ==Applications == | ||
Applications are programs that often provide a GUI and/or use other toolkits, e.g. [http://vtk.org/ VTK]. | Applications are programs that often provide a GUI and/or use other toolkits, e.g. [http://vtk.org/ VTK]. | ||
===Process=== | ===Process=== | ||
Applications | Applications have their own ''CMakeLists.txt'' files. They will either locate ITK or build it as part of their process. | ||
* Alternatives | * Alternatives | ||
** Use the CMake ''find_package(ITK)'' command from the application. | ** Use the CMake ''find_package(ITK)'' command from the application. | ||
Line 140: | Line 140: | ||
== Bug Fixes == | == Bug Fixes == | ||
Bug fixes are contributions that repair defects reported in the [http:// | Bug fixes are contributions that repair defects reported in the [http://issues.itk.org JIRA Issue Tracker] or repair daily defects that creep into ITK. | ||
===Process=== | ===Process=== | ||
# Follow [[ITK/Git/Develop|the procedure to become an ITK developer]]. | # Follow [[ITK/Git/Develop|the procedure to become an ITK developer]]. | ||
# To reference JIRA issue XY, add " (#ITK-XY)" to the END of the FIRST line." or '''BUG: ITK-XY''' as the commit prefix. | |||
# Post as a [[ITK/Git/Develop#Workflow|Gerrit Patch]] | # Post as a [[ITK/Git/Develop#Workflow|Gerrit Patch]] | ||
## Gerrit provides a Peer-review mechanism for the patch | ## Gerrit provides a Peer-review mechanism for the patch | ||
## Gerrit uses [[ITK_Release_4/Testing_On_Demand/Tutorial|Cdash@home]] to provide automated testing of patches | ## Gerrit uses [[ITK_Release_4/Testing_On_Demand/Tutorial|Cdash@home]] to provide automated testing of patches | ||
# After Gerrit peer-review, [[ITK/Git/Develop#Merge_a_Topic|merge the topic]]. | # After Gerrit peer-review approval, [[ITK/Git/Develop#Merge_a_Topic|merge the topic]]. | ||
==Wiki Examples== | ==Wiki Examples== | ||
Wiki examples are independent, compilable examples that illustrate ITK concepts. | Wiki examples are independent, compilable examples that illustrate ITK concepts. | ||
'''Please Note:''' The examples illustrate existing ITK classes and '''do not''' introduce new classes. | |||
===Process=== | ===Process=== | ||
Follow [[ITK/Examples/Instructions/ForDevelopers|this procedure]] to add a Wiki example. | Follow [[ITK/Examples/Instructions/ForDevelopers|this procedure]] to add a Wiki example. | ||
= ITKv4 Refactoring = | = ITKv4 Refactoring = | ||
Line 180: | Line 175: | ||
* New Classes | * New Classes | ||
* Modified existing classes | * Modified existing classes | ||
* [https://github.com/picslITK/ITK4_topics/tree/RegistrationRefactoringNew Registration Refactory github] - Registration New branch] | |||
== Pipeline Refactoring == | == Pipeline Refactoring == | ||
Line 241: | Line 237: | ||
* New Classes | * New Classes | ||
* New Internal | ** Particle System Data Container | ||
** Cost Functions | |||
** Optimizers | |||
* New Internal Module (about 12 classes) | |||
** Particle System Module | |||
*** Higher level filters | |||
**** Longitudinal shape modeling pipeline (Filter) | |||
**** Procrustes Registration (Affine transform output) | |||
* SVN Repository | |||
==[[ITK_Release_4/The_Team/A2D2_Development_Team#3D_Real-time_Physics-based_Non-rigid_Registration_for_Image-guided_Neurosurgery|3D Real-time Physics-based Non-rigid Registration for Image-guided Neurosurgery]]== | ==[[ITK_Release_4/The_Team/A2D2_Development_Team#3D_Real-time_Physics-based_Non-rigid_Registration_for_Image-guided_Neurosurgery|3D Real-time Physics-based Non-rigid Registration for Image-guided Neurosurgery]]== |
Latest revision as of 15:59, 9 December 2011
This page describes the multiple mechanisms available to contribute code to the ITK Ecosystem.
Overview
Level of Contribution
New Classes
A new class must be submitted to the Insight Journal.
Please Note: This requirement is waived for ITKv4 contractors.
Please Note: ITKv4 will remove the Review directory. Moving classes from Review into the appropriate Module must follow the process for adding new classes. This document describes the destination Module for each class in Review.
Process
- Follow the procedure to become an ITK developer and setup your environment.
- Submit an article to the Insight Journal
- Waived for ITKv4 contractors.
- Follow the procedure to add a new class.
- Review the code using the Checklist
- Post as a Gerrit Patch
- Gerrit provides a Peer-review mechanism for the patch
- Gerrit uses Cdash@home to provide automated testing of patches
- After Gerrit peer-review approval, merge the topic.
Internal Modules
Internal modules are full fledged modules that will be integrated with the ITK git repository and will be distributed as part of ITK.
Process
- Follow the procedure to become an ITK developer and setup your environment.
- Follow the procedure to Add an Internal Module
- Create a topic branch
- Review the code using the Checklist
- For review purposes:
- Rebase topic branch on top of master
- git checkout -b topicbranchforreview topicbranch
- git rebase origin/master
- Squash topic branch and submit to Gerrit
- The patch in Gerrit is not intended to be merged directly into ITK. It is here ONLY to be reviewed.
- Rebase topic branch on top of master
- When approved in Gerrit:
- Go back to topic branch
- Merge master into topic branch
- Notify ITK developers list, requesting your branch to be merged into ITK master
Examples
- Video classes
- GPU Support
- Deconvolution
Bridge Modules
A bridge module depends on third party headers not distributed with ITK. A bridge module usually connects classes that interface to two or more toolkits.
Process
- Follow the procedure to become an ITK developer and setup your environment.
- Follow the procedure to Add an Internal Module.
- Place your module in the group Bridge
- Create itk-module-init.cmake
- This file should contain find_package commands to locate the third party libraries and headers
- Edit the itk-module.cmake and add EXCLUDE_FROM_ALL to the itk_module call. This prevents this Bridge Module from being included when ITK_BUILD_ALL_MODULES is selected during the cmake configuration. To build your module you will need to explicitly turn it on. For example, if your Bridge Module is called Foo, then while running cmake a Module_ITK-Foo variable will be listed under the Modules group
- Create a topic branch, add and commit your changes.
- Review the code using the Checklist
- For review purposes:
- Rebase topic branch on top of master
- git checkout -b topicbranchforreview topicbranch
- git rebase origin/master
- Squash topic branch and submit to Gerrit
- The patch in Gerrit is not intended to be merged directly into ITK. It is here ONLY to be reviewed.
- Rebase topic branch on top of master
- When approved in Gerrit:
- Go back to topic branch
- Merge master into topic branch
- Notify ITK developers list, requesting your branch to be merged into ITK master
Examples
- Connecting ITK to VTK
- Connecting ITK to OpenCV
- Connecting ITK to dcmtk
External Modules
External modules are ITK-like modules that cannot be included in ITK because of license issues or should not be included in ITK because of scope and/or specialized use.
Process
- Follow the procedure to become an ITK developer and setup your environment.
- Follow the procedure to Add an External Module
Examples
- Lesion Sizing Toolkit
- see the Lesion Sizing Toolkit.
- PETSc
- Numerical Libraries that too large to be included in ITK proper
- FFTW
- Library distributed under GPL (overriding the ITK's Apache License)
External Layers
Software that presents a different interface to ITK.
Process
- Alternatives
- Use the CMake find_package(ITK) command from the external layer (library or application).
- Use ITK as an external project in a Superbuild approach.
- The Superbuild groups the external layer along with ITK into a larger distribution...
Examples
- SimpleITK
- ManagedITK
- Wrapping for C#
Applications
Applications are programs that often provide a GUI and/or use other toolkits, e.g. VTK.
Process
Applications have their own CMakeLists.txt files. They will either locate ITK or build it as part of their process.
- Alternatives
- Use the CMake find_package(ITK) command from the application.
- Use ITK as a CMake external project in a Superbuild approach.
Examples
- find_package
- Volview Plugins
- V3D Plugins
- ITK Wiki Examples
- Superbuild
New Data
Data may be needed as part of testing code and/or examples on how to run new classes, applications or layers.
Process
Please see the instructions on how to add data.
Bug Fixes
Bug fixes are contributions that repair defects reported in the JIRA Issue Tracker or repair daily defects that creep into ITK.
Process
- Follow the procedure to become an ITK developer.
- To reference JIRA issue XY, add " (#ITK-XY)" to the END of the FIRST line." or BUG: ITK-XY as the commit prefix.
- Post as a Gerrit Patch
- Gerrit provides a Peer-review mechanism for the patch
- Gerrit uses Cdash@home to provide automated testing of patches
- After Gerrit peer-review approval, merge the topic.
Wiki Examples
Wiki examples are independent, compilable examples that illustrate ITK concepts.
Please Note: The examples illustrate existing ITK classes and do not introduce new classes.
Process
Follow this procedure to add a Wiki example.
ITKv4 Refactoring
FEM Refactoring
- Modified existing classes
Level Sets Refactoring
- New Classes
- Modified existing classes
Registration Refactoring
- New Classes
- Modified existing classes
- Registration Refactory github - Registration New branch]
Pipeline Refactoring
- ??
GPU
- New classes
- New Internal Module
DICOM
- GDCM level ?
- Integrated with GDCM upstream ?
SimpleITK
- External Layer
WrapITK
- Internal Layer
ITKv4 A2D2 Contributions
A Comprehensive Workflow for Robust Characterization of Microstructure for Cancer Studies
- New Application (with GUI and Viz)
- New classes (denoising)
Comprehensive Workflow for Large Histology Segmentation and Visualization
- New classes
- Color correction, artifact removal, image registration
Adding Deconvolution Algorithms to ITK
- New classes
- New Internal Module
ITK Extensions for Video Processing
- New Internal Modules
- Some Bridge Modules
Real-Time Image Capture for ITK through a Video Grabber
- New Classes
- Some Bridge Modules
Methods in Medical Image Analysis: an ITK-based course
- New Classes
- Some Bridge Modules
- (OpenCV point cloud library)
ITK Algorithms for Analyzing Time-varying Shape with Application to Longitudinal Heart Modeling
- New Classes
- Particle System Data Container
- Cost Functions
- Optimizers
- New Internal Module (about 12 classes)
- Particle System Module
- Higher level filters
- Longitudinal shape modeling pipeline (Filter)
- Procrustes Registration (Affine transform output)
- Higher level filters
- Particle System Module
- SVN Repository
3D Real-time Physics-based Non-rigid Registration for Image-guided Neurosurgery
- Application
- External Layer ?
Denoising Microscopy, MRI, and Ultrasound Images
- New Classes
- New Internal Modules