Contribution
Setting up your environment for development
Fork the repository from github and clone your fork locally (see here to setup your ssh key):
git clone git@github.com:<your_username>/wonkyconn.git
Set up a development environment. We recommend using pixi:
pixi install
Alternatively, you can use a virtual environment with pip:
python3 -m venv wonkyconn
source wonkyconn/bin/activate
pip install -e ".[dev]"
Install the data required for testing.
The test data is managed with datalad. After installing datalad, fetch the test data:
datalad get wonkyconn/data/test_data
Install pre-commit hooks to run all the checks before each commit.
pre-commit install
Contributing to code
This is a very generic workflow.
Comment on an existing issue or open a new issue referencing your addition.
Tip
Review and discussion on new code can begin well before the work is complete, and the more discussion the better! The development team may prefer a different path than you’ve outlined, so it’s better to discuss it and get approval at the early stage of your work.
On your fork, create a new branch from main:
git checkout -b your_branch
Make the changes, lint, and format.
Commit your changes on this branch.
If you want to make sure all the tests will be run by github continuous integration,
make sure that your commit message contains full_test.
Run the tests locally; you can run specific tests to speed up the process:
pytest -v wonkyconn/tests/test_connectome.py::test_calculate_intranetwork_correlation
push your changes to your online fork. If this is the first commit, you might want to set up the remote tracking:
git push origin HEAD --set-upstream
In the future you can simply do:
git push
Submit a pull request from your fork of the repository.
Check that all continuous integration tests pass.
Contributing to documentation
The workflow is the same as code contributions, with some minor differences.
Install the
docsdependencies.
Using pixi:
pixi run -e docs sphinx-build docs/source docs/build/html
Or with pip:
pip install -e ".[docs]"
cd docs
make html
After making changes, review the generated HTML locally.
Submit your changes.
Writing a PR
When opening a pull request, please use one of the following prefixes:
[ENH] for enhancements
[FIX] for bug fixes
[TEST] for new or updated tests
[DOCS] for new or updated documentation
[STYL] for stylistic changes
[MAINT] for refactoring existing code, any maintainace related things
Pull requests should be submitted early and often (please don’t mix too many unrelated changes within one PR)! If your pull request is not yet ready to be merged, please submit as a drafted PR. This tells the development team that your pull request is a “work-in-progress”, and that you plan to continue working on it.
One your PR is ready a member of the development team will review your changes to confirm that they can be merged into the main codebase.
Running the demo
You can run a demo by downloading some test data.
Run the following from the root of the repository.
datalad get wonkyconn/data/test_data
INPUT=wonkyconn/data/test_data/connectome_Schaefer2018
OUTPUT=wonkyconn/data/test_data/test_output
ATLAS_PATH=wonkyconn/data/test_data/connectome_Schaefer2018/atlases/sub-1/func/sub-1_seg-Schaefer2018400Parcels7Networks_dseg.nii.gz
PHENOTYPE=wonkyconn/data/test_data/connectome_Schaefer2018/participants.tsv
wonkyconn ${INPUT} ${OUTPUT} group \
--phenotypes ${PHENOTYPE} \
--atlas Schaefer2018400Parcels7Networks ${ATLAS_PATH} \
--group-by seg desc \
--verbosity 2
Prepare a release
Currently this project is not pushed to PyPi. We simply tag the version on the repository so users can reference the version of installation. The release process will trigger a new tagged docker build of the software.
Switch to a new branch locally:
git checkout -b REL-x.y.z
First we need to prepare the release by updating the file wonkyconn/docs/changes.md to make sure all the new features, enhancements, and bug fixes are included in their respective sections.
Finally, we need to change the title from x.y.z.dev to x.y.z
## x.y.z
**Released MONTH YEAR**
### New
...
Add these changes and submit a PR:
git add docs/
git commit -m "REL x.y.z"
git push upstream REL-x.y.z
Once the PR has been reviewed and merged, pull from master and tag the merge commit:
git checkout main
git pull upstream main
git tag x.y.z
git push upstream --tags
Post-release
At this point, the release has been made.
We also need to create a new section in wonkyconn/docs/changes.md with a title and the usual New, Enhancements, Bug Fixes, and Changes sections for the version currently under development:
## x.y.z+1.dev
**Released MONTH YEAR**
### New
### Fixes
### Enhancements
### Changes
Based on contributing guidelines from the STEMMRoleModels project and Nilearn contribution guidelines.