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

From KitwarePublic
< ITK‎ | Release 4
Jump to navigationJump to search
Line 148: Line 148:
## 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==

Revision as of 00:18, 26 June 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 Mantis Bug Tracker or repair daily defects that creep into ITK.

Process

  1. Follow the procedure to become an ITK developer.
  2. To reference Mantis issue XY, add " (#XY)" to the END of the FIRST line." or BUG: #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

  • New Classes
  • Modified existing classes

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
  • New Internal Modules

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