Skip to content

Latest commit

 

History

History
328 lines (269 loc) · 12.3 KB

CONTRIBUTING.md

File metadata and controls

328 lines (269 loc) · 12.3 KB
Raw

Development Model

Development generally follows the following ideas:

  • New features are merged into to the development branch using Pull Requests (PRs).

    Regression testing is used to ensure that no answers change (or if they do, that the changes were expected).

  • Bug fixes, questions and contributions of new features are welcome!

    • Bugs should be reported through GitHub Issues.

    • We suggest asking questions through GitHub Discussions.

    • All contributions should be done via pull requests. A pull request should be generated from your fork of REMORA and target the development branch. See below for details on how this process works.

      In general we squash commits upon merge to have a clean history. Please ensure that your PR title and first post are descriptive, since these will be used for a squashed commit message.

      Please note the following: If you choose to make contributions to the code then you hereby grant a non-exclusive, royalty-free perpetual license to install, use, modify, prepare derivative works, incorporate into other computer software, distribute, and sublicense such enhancements or derivative works thereof, in binary and source code form.

Git workflow

REMORA uses git for version control. If you are new to git, you can follow one of these tutorials:

Make your own fork and create a branch on it

The basic workflow is:

  • Fork the main repo (or update it if you already created it).
  • Implement your changes and push them on a new branch <branch_name> on your fork.
  • Create a Pull Request from branch <branch_name> on your fork to branch development on the main REMORA repository.

First, let us setup your local git repo. To make your own fork of the main (upstream) repository, press the fork button on the REMORA Github page.

Then, clone your fork on your local computer. If you plan on doing a lot of REMORA development, we recommend configuring your clone to use ssh access so you won't have to enter your Github password every time, which you can do using these commands:

git clone --branch development git@github.com:<myGithubUsername>/REMORA.git

# Then, navigate into your repo, add a new remote for the main REMORA repo, and fetch it:
cd REMORA
git remote add upstream https://github.com/seahorce-scidac/REMORA
git remote set-url --push upstream git@github.com:<myGithubUsername>/REMORA.git
git fetch upstream

# We recommend setting your development branch to track the upstream one instead of your fork:
git branch -u upstream/development

For instructions on setting up SSH access to your Github account on a new machine, see here.

If you instead prefer to use HTTPS authentication, configure your local clone as follows:

git clone --branch development https://github.com/<myGithubUsername>/REMORA.git

# Navigate into your repo, add a new remote for the main REMORA repo, and fetch it
cd REMORA
git remote add upstream https://github.com/REMORA
git remote set-url --push upstream https://github.com/<myGithubUsername>/REMORA.git
git fetch upstream

# We recommend setting your development branch to track the upstream one instead of your fork:
git branch -u upstream/development

Now you are free to play with your fork (for additional information, you can visit the Github fork help page).

Note: you do not have to re-do the setup above every time. Instead, in the future, you need to update the development branch on your fork with

git checkout development
git pull

Make sure you are on the development branch with

git checkout development

in the REMORA directory.

Create a branch <branch_name> (the branch name should reflect the piece of code you want to add, like add_new_physics) with

git checkout -b <branch_name>

and do the coding you want. Add the files you work on to the git staging area with

git add <file_I_created> <and_file_I_modified>

Commit & push your changes

Periodically commit your changes with

git commit -m "This is a 50-char description to explain my work"

The commit message (between quotation marks) is super important in order to follow the developments and identify bugs.

For the moment, commits are on your local repo only. You can push them to your fork with

git push -u origin <branch_name>

If you want to synchronize your branch with the development branch (this is useful when development is being modified while you are working on <branch_name>), you can use

git pull upstream development

and fix any conflicts that may occur.

Do not merge your branch for PR into your local development branch, because it will make your local development branch diverge from the matching branch in the main repository after your PR is merged.

Submit a Pull Request

A Pull Request is the way to efficiently visualize the changes you made and to propose your new feature/improvement/fix to the REMORA project. Right after you push changes, a banner should appear on the Github page of your fork, with your <branch_name>.

  • Click on the compare & pull request button to prepare your PR.
  • It is time to communicate your changes: write a title and a description for your PR. People who review your PR are happy to know
    • what feature/fix you propose, and why
    • how you made it (created a new class than inherits from...)
    • and anything relevant to your PR (performance tests, images, etc.)
  • Press Create pull request. Now you can navigate through your PR, which highlights the changes you made.

Please DO NOT write large Pull Requests, as they are very difficult and time-consuming to review. As much as possible, split them into small, targeted PRs. For example, if find typos in the documentation open a pull request that only fixes typos. If you want to fix a bug, make a small pull request that only fixes a bug. If you want to implement a large feature, write helper functionality first, test it and submit those as a first pull request. If you want to implement a feature and are not too sure how to split it, just open a discussion about your plans and ping other REMORA developers on it to chime in.

Even before your work is ready to merge, it can be convenient to create a PR (so you can use Github tools to visualize your changes). In this case, please make a "draft" PR using the drop-down menu next to the "Create pull request" button.

Once your pull request is made, we will review and potentially merge it. We recommend always creating a new branch for each pull request, as per the above instructions. Once your pull request is merged, you can delete your local PR branch with

git branch -D <branch_name>

and you can delete the remote one on your fork with

git push origin --delete <branch_name>

Generally speaking, you want to follow the following rules.

  • Do not merge your branch for PR into your local development branch that tracks REMORA development branch. Otherwise your local development branch will diverge from REMORA development branch.

  • Do not commit in your development branch that tracks REMORA development branch.

  • Always create a new branch based off development branch for each pull request, unless you are going to use git to fix it later.

If you have accidentally committed in development branch, you can fix it as follows,

git checkout -b new_branch
git checkout development
git reset HEAD~2  # Here 2 is the number of commits you have accidentally committed in development
git checkout .

After this, the local development should be in sync with REMORA development and your recent commits have been saved in new_branch branch.

If for some reason your PR branch has diverged from REMORA, you can try to fix it as follows. Before you try it, you should back up your code in case things might go wrong.

git fetch upstream   # assuming upstream is the remote name for the official REMORA repo
git checkout -b xxx upstream/development  # replace xxx with whatever name you like
git branch -D development
git checkout -b development upstream/development
git checkout xxx
git merge yyy  # here yyy is your PR branch with unclean history
git rebase -i upstream/development

You will see something like below in your editor,

pick 7451d9d commit message a
pick c4c2459 commit message b
pick 6fj3g90 commit message c

This now requires a bit of knowledge on what those commits are, which commits have been merged, which commits are actually new. However, you should only see your only commits. So it should be easy to figure out which commits have already been merged. Assuming the first two commits have been merged, you can drop them by replace pick with drop,

drop 7451d9d commit message a
drop c4c2459 commit message b
pick 6fj3g90 commit message c

After saving and then exiting the editor, git log should show a clean history based on top of development branch. You can also do git diff yyy..xxx to make sure nothing new was dropped. If all goes well, you can submit a PR using xxx branch. Don't worry, if something goes wrong during the rebase, you an always git rebase --abort and start over.

REMORA Coding Style Guide

Code Guidelines

REMORA developers should adhere to the following coding guidelines:

  • Indentations should use 4 spaces, not tabs.
  • Use curly braces for single statement blocks. For example:
       for (int n=0; n<10; ++n) {
           Print() << "Like this!";
       }

or

       for (int n=0; n<10; ++n) { Print() << "Like this!"; }

but not

       for (int n=0; n<10; ++n) Print() << "Not like this.";

or

       for (int n=0; n<10; ++n)
          Print() << "Not like this.";
  • Add a space after the function name and before the parenthesis of the parameter list (but not when simply calling the function). For example:
        void CorrectFunctionDec (int input)

Not

        void IncorrectFunctionDec(int input)

This makes it easy to find where functions are defined with grep.

  • Member variables should be prefixed with m_. For example:
       amrex::Real m_variable;

These guidelines should be adhered to in new contributions to REMORA, but please refrain from making stylistic changes to unrelated sections of code in your PRs.

Fixing Style Issues

On any pull request, or any new commits to an open pull request that trigger the REMORA CI tests, REMORA will run style checks. These checks ensure the developer is using spaces instead of tabs and has not inserted any whitespace at the end of source lines.

We require pull requests to pass these style checks before merging. If failures arise, there is an easy way to automatically fix style issues like this by running the following shell commands from the main REMORA repository.

For removing whitespace at the end of lines:

$ .github/workflows/style/check_trailing_whitespaces.sh

For replacing tabs with 4 spaces:

$ .github/workflows/style/check_tabs.sh

These commands will modify your local source files, so it is best to run these commands after committing your local changes and then add and commit the style changes that result.

These commands will also output a git diff that shows the changes made by the style scripts.

API Documentation Using Doxygen

The Doxygen documentation is designed for advanced user-developers. It aims to maximize the efficiency of a search-and-find style of locating information. Doxygen style comment blocks should proceed the namespace, class, function, etc. to be documented where appropriate. For example:

   /**
    * \brief A one line description.
    *
    * \param[in] variable A short description of the variable.
    * \param[inout] data The value of data is read and changed.
    *
    * A longer description can be included here.
    */

    void MyFunction (int variable, MultiFab& data){
    ...

Additional information regarding Doxygen comment formatting can be found in the Doxygen Manual.