ITK/Release 4/New Code Contribution Process: Difference between revisions

From KitwarePublic
< ITK‎ | Release 4
Jump to navigationJump to search
 
(90 intermediate revisions by 3 users not shown)
Line 5: Line 5:
= Level of Contribution =
= Level of Contribution =


== Bug Fixes ==
== New Classes==
Bug fixes are contributions that repair defects reported in the [http://public.kitware.com/Bug/view_all_bug_page.php Mantis Bug Tracker] or repair daily defects that creep into ITK.
A new class must be submitted to the [http://www.insight-journal.org/ Insight Journal].
# Follow [[ITK/Git/Develop|the procedure to become an ITK developer]].
 
'''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. [https://spreadsheets.google.com/spreadsheet/ccc?key=0ApfVzIsZe0VFdFUxdE95WE5kMVE4LTJ6b3ZYQnpRalE&hl=en_US&ndplr=1#gid=0 This document] describes the destination Module for each class in Review.
 
 
===Process===
# Follow [[ITK/Git|the procedure to become an ITK developer and setup your environment]].
# [http://www.insight-journal.org/help/submission Submit an article to the Insight Journal]
#* Waived for ITKv4 contractors.
# Follow [[ITK_Release_4/Modularization/Add_new_classes|the procedure to add a new class]].
# Review the code using the [[ITK_Release_4/Modularization/Code Reviews/Checklist|Checklist]]
# 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]].


== New Classes and Filters ==
== Internal Modules ==
A new class (or filter) must be submitted to the [http://www.insight-journal.org/ Insight Journal].
# [http://www.insight-journal.org/help/submission Submission to the Insight Journal]
# Code Reviewed using the [[ITK_Release_4/Modularization/Code Reviews/Checklist|Checklist]]
# Posted as Gerrit Patches
# Peer-reviewed
# Cdash@home tested
# Merged


== 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 [[ITK/Git|the procedure to become an ITK developer and setup your environment]].
# Follow [[ITK_Release_4/Modularization/_Add_a_module| the procedure to Add an Internal Module]]
# Create a topic branch
# Review the code using the [[ITK_Release_4/Modularization/Code Reviews/Checklist|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.
# 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 [[ITK/Git|the procedure to become an ITK developer and setup your environment]].
# Follow [[ITK_Release_4/Modularization/_Add_a_module| 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 [[ITK_Release_4/Modularization/Code Reviews/Checklist|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.
# 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 ==
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. For an example see the [http://public.kitware.com/LesionSizingKit/index.php/Main_Page Lesion Sizing Toolkit].
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 [[ITK/Git|the procedure to become an ITK developer and setup your environment]].
# Follow [[ITK_Release_4/Modularization/Add_an_external_module_%28external_module%29|the procedure to Add an External Module]]
 
===Examples===
* Lesion Sizing Toolkit
** see the [http://public.kitware.com/LesionSizingKit/index.php/Main_Page 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 ==
== External Layers ==
Software that presents a different interface to ITK. For example see [[ITK_Release_4/SimpleITK|SimpleITK]].


== Applications ==
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===
* [[ITK_Release_4/SimpleITK|SimpleITK]]
* ManagedITK
** 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].


== Wiki Examples ==
===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 [http://www.kitware.com/products/html/BuildingExternalProjectsWithCMake2.8.html CMake external project] in a Superbuild approach.
 
===Examples===
 
* find_package
** Volview Plugins
** V3D Plugins
** [[ITK/Examples|ITK Wiki Examples]]
 
* Superbuild
** [http://www.slicer.org/slicerWiki/index.php/Slicer4 Slicer4]
 
== 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 [[ITK/Git/Develop/Data|instructions on how to add data]].
 
== Bug Fixes ==
 
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===
# 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]]
## 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
# After Gerrit peer-review approval, [[ITK/Git/Develop#Merge_a_Topic|merge the topic]].
 
==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===
Follow [[ITK/Examples/Instructions/ForDevelopers|this procedure]] to add a Wiki example.
Follow [[ITK/Examples/Instructions/ForDevelopers|this procedure]] to add a Wiki example.


'''NOTE:''' The examples illustrate existing ITK classes and do not introduce new classes.
= ITKv4 Refactoring =
 
[[ITK_Release_4/The Team/ITKv4 Development Team|ITKv4 Development Team]]
 
== FEM Refactoring ==
 
* Modified existing classes
 
== Level Sets Refactoring ==
 
* New Classes
* Modified existing classes
 
== Registration Refactoring ==
 
* New Classes
* Modified existing classes
* [https://github.com/picslITK/ITK4_topics/tree/RegistrationRefactoringNew 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 =
 
[[ITK_Release_4/The Team/A2D2 Development Team|A2D2 Development Team]]
 
== [[ITK_Release_4/The_Team/A2D2_Development_Team#A_Comprehensive_Workflow_for_Robust_Characterization_of_Microstructure_for_Cancer_Studies|A Comprehensive Workflow for Robust Characterization of Microstructure for Cancer Studies]] ==
 
* New Application (with GUI and Viz)
* New classes (denoising)
 
== [[ITK_Release_4/The_Team/A2D2_Development_Team#Comprehensive_Workflow_for_Large_Histology_Segmentation_and_Visualization|Comprehensive Workflow for Large Histology Segmentation and Visualization]]  ==
 
* New classes
** Color correction, artifact removal, image registration
 
== [[ITK_Release_4/The_Team/A2D2_Development_Team#Adding_Deconvolution_Algorithms_to_ITK|Adding Deconvolution Algorithms to ITK]] ==
 
* New classes
* New Internal Module
 
==[[ITK_Release_4/The_Team/A2D2_Development_Team#ITK_Extensions_for_Video_Processing|ITK Extensions for Video Processing]]==
 
* New Internal Modules
** Some Bridge Modules
 
==[[ITK_Release_4/The_Team/A2D2_Development_Team#Real-Time_Image_Capture_for_ITK_through_a_Video_Grabber|Real-Time Image Capture for ITK through a Video Grabber]]==
 
* New Classes
* Some Bridge Modules
 
==[[ITK_Release_4/The_Team/A2D2_Development_Team#Methods_in_Medical_Image_Analysis:_an_ITK-based_course|Methods in Medical Image Analysis: an ITK-based course]]==
 
* New Classes
* Some Bridge Modules
** (OpenCV point cloud library)
 
==[[ITK_Release_4/The_Team/A2D2_Development_Team#ITK_Algorithms_for_Analyzing_Time-varying_Shape_with_Application_to_Longitudinal_Heart_Modeling|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)
 
* 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]]==
 
* Application
** External Layer ?
 
==[[ITK_Release_4/The_Team/A2D2_Development_Team#Denoising_Microscopy.2C_MRI.2C_and_Ultrasound_Images|Denoising Microscopy, MRI, and Ultrasound Images]]==
 
* New Classes
* New Internal Modules
 
==[[ITK_Release_4/The_Team/A2D2_Development_Team#Framework_for_Automated_Parameter_Tuning_of_ITK_Registration_Pipelines|Framework for Automated Parameter Tuning of ITK Registration Pipelines]]==
 
==[[ITK_Release_4/The_Team/A2D2_Development_Team#SCORE:_Systematic_Comparison_through_Objective_Rating_and_Evaluation|SCORE: Systematic Comparison through Objective Rating and Evaluation]]==
 
==[[ITK_Release_4/The_Team/A2D2_Development_Team#SCORE.2B.2B:_Crowd_sourced_data.2C_automatic_segmentation.2C_and_ground_truth_for_ITK4|SCORE++: Crowd sourced data, automatic segmentation, and ground truth for ITK4]]==
 
==[[ITK_Release_4/The_Team/A2D2_Development_Team#Fostering_Open_Science_for_Lung_Cancer_Lesion_Sizing|Fostering Open Science for Lung Cancer Lesion Sizing]]==

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

  1. Follow the procedure to become an ITK developer and setup your environment.
  2. Submit an article to the Insight Journal
    • Waived for ITKv4 contractors.
  3. Follow the procedure to add a new class.
  4. Review the code using the Checklist
  5. Post as a Gerrit Patch
    1. Gerrit provides a Peer-review mechanism for the patch
    2. Gerrit uses Cdash@home to provide automated testing of patches
  6. 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

  1. Follow the procedure to become an ITK developer and setup your environment.
  2. Follow the procedure to Add an Internal Module
  3. Create a topic branch
  4. Review the code using the Checklist
  5. 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.
  6. 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

  1. Follow the procedure to become an ITK developer and setup your environment.
  2. 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
  3. Create a topic branch, add and commit your changes.
  4. Review the code using the Checklist
  5. 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.
  6. 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

  1. Follow the procedure to become an ITK developer and setup your environment.
  2. Follow the procedure to Add an External Module

Examples

  • 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

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

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

  1. Follow the procedure to become an ITK developer.
  2. To reference JIRA issue XY, add " (#ITK-XY)" to the END of the FIRST line." or BUG: ITK-XY as the commit prefix.
  3. Post as a Gerrit Patch
    1. Gerrit provides a Peer-review mechanism for the patch
    2. Gerrit uses Cdash@home to provide automated testing of patches
  4. 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

ITKv4 Development Team

FEM Refactoring

  • Modified existing classes

Level Sets Refactoring

  • New Classes
  • Modified existing classes

Registration Refactoring

Pipeline Refactoring

  • ??

GPU

  • New classes
  • New Internal Module

DICOM

  • GDCM level ?
    • Integrated with GDCM upstream ?

SimpleITK

  • External Layer

WrapITK

  • Internal Layer

ITKv4 A2D2 Contributions

A2D2 Development Team

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)
  • 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

Framework for Automated Parameter Tuning of ITK Registration Pipelines

SCORE: Systematic Comparison through Objective Rating and Evaluation

SCORE++: Crowd sourced data, automatic segmentation, and ground truth for ITK4

Fostering Open Science for Lung Cancer Lesion Sizing