Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.
Comment: Final iteration – attempted to bring it to higher level

The goal of the ACME Code Development Process is to achieve high developer productivity towards the improvement of the ACME earth system model. A degree of formality is needed because of the large size of the ACME team, the geographic distribution of the team, and large scope of the ACME project, particularly when including collaborations.

The process is meant to promote the quality of the ACME model along many dimensions: high-quality science, verified implementations, high-performing implementations, high-quality code, portability to DOE computer architectures, and maintainability/extensibility by those other then the original developers. The process intends to give individuals and small teams the ability to independently develop new features and algorithms, yet with a clear path to incorporation into the ACME model. 

Steps Main steps of the Development ProcessACME code development process:

  1. Code development should begin from the head of the master branch or most recent release branch of ACME from the github repository, and take place on a feature branch
  2. Code should be of high enough quality to be readable, modifiable, and maintainable by others
  3. New code in an existing routine should follow the style of the surrounding code.
  4. Brand new code should follow the recommended ACME coding standards (TBD
  5. Code development should follow ACME standards for using git, e.g. preparing a separate commit for each stand-alone contribution along with a well-formed commit message.
  6. Developers must run the ACME test suite to verify that their development has not unintentionally changed the code behavior, and all new development must be accompanied by tests to protect the new feature against future developments.
  7. Code development should be incorporated into, or rebased to, the ACME code base at least every few months (since ACME has no resources to reconcile code that has been on diverging development paths).
  8. For eventual inclusion in of a new feature or algorithm into the ACME modelModel, documentation of the development should document the main steps of the ACME Code Review process should be created:
    1. A design document (or paper) detailing the equations/algorithms that are being implemented
    2. Verification evidence that supports that the implementation is correct
    3. One or more tests added to the ACME testing system to protect the new feature against future developments
    4. Performance analysis and data showing the expected and measured performance impact of the new feature
    5. Validation evidence that the feature matches observational data
  9. Code development should be incorporated into, or rebased to, the ACME code base at least every few months, and developers must verify that the ACME tests still pass on this development branch (ACME has no resources to reconcile code that has been on diverging development paths).
  10. Code development should follow ACME git standards:
    1. Good commit messages (example:  http://chris.beams.io/posts/git-commit/)
    2. Clean commit history: Each commit should be a stand-alone self describing entity. Ideally each commit should be testable as well.

...