Skip to content

Latest commit

 

History

History
69 lines (47 loc) · 2.83 KB

README.md

File metadata and controls

69 lines (47 loc) · 2.83 KB

GitHub Workflow Status GitHub Workflow Status

HiRes-CESM Analysis

This repository is building a set of tools for analyzing BGC output in a high-resolution POP run.

For Developers

A few recommended practices to incorporate in your development sandbox:

Keep your conda environment up to date

The first time you check out this repository, run

$ conda env install -f environments/environment.yaml

If you notice the YAML file has changed after you fetch changes from github, update the environment with

$ conda env update -f environments/environment.yaml

If the env update command fails, you can remove the environment and re-create it:

$ conda env remove --name hires-marbl
$ conda env create -f environments/environment.yaml

Use pre-commit to test code before commiting

Please take advantage of the pre-commit package to ensure that black is run before commiting:

$ pre-commit install --install-hooks # set up pre-commit
$ pre-commit run -a                  # check all the files currently in the repo

The pre-commit package is already installed via the hires-marbl conda environment. There is a github action to run these checks on all pull requests, but running them locally via-pre-commit will reduce the number of failed actions. NOTE: for some reason, to properly install pre-commit on the CISL systems, the above command must be run from casper rather than cheyenne.

Note that pre-commit creates a virtual environment using specific tags of each package. As newer versions of black become available on conda-forge, we will update the pre-commit environment.

Run pytest after modifying python in utils/

To test some of the python code in notebooks/utils/, run pytest. These tests can be run from the top level of this repository by running

$ pytest tests/

If you add new code to this directory, consider writing small tests to ensure it is running as expected.