-
Notifications
You must be signed in to change notification settings - Fork 5
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #13 from 62442katieb/fixed-nbs
Added residualizing and scaling to NBS-Predict
- Loading branch information
Showing
34 changed files
with
6,744 additions
and
815 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,125 @@ | ||
# Contributing to IDConn | ||
|
||
Welcome to the ``IDConn`` repository! | ||
We're excited you're here and want to contribute. | ||
|
||
These guidelines are designed to make it as easy as possible to get involved. | ||
If you have any questions that aren't discussed below, please let us know by opening an [issue][link_issues]! | ||
|
||
Before you start you'll need to set up a free [GitHub][link_github] account and sign in. | ||
Here are some [instructions][link_signupinstructions]. | ||
|
||
## Governance | ||
|
||
Governance is a hugely important part of any project. | ||
It is especially important to have clear process and communication channels for open source projects that rely on a distributed network of volunteers, such as ``IDConn``. | ||
|
||
``IDConn`` is currently supported by a small group of core developers. | ||
Even with only a couple of individuals involved in decision making processes, we've found that setting expectations and communicating a shared vision has great value. | ||
|
||
By starting the governance structure early in our development, we hope to welcome more people into the contributing team. | ||
We are committed to continuing to update the governance structures as necessary. | ||
Every member of the ``IDConn`` community is encouraged to comment on these processes and suggest improvements. | ||
|
||
As the first project leader, Katie Bottenhorn is ultimately responsible for any major decisions pertaining to ``IDConn`` development. | ||
However, all potential changes are explicitly and openly discussed in the described channels of communication, and we strive for consensus amongst all community members. | ||
|
||
## Code of conduct | ||
|
||
All ``IDConn`` community members are expected to follow our [code of conduct](https://github.com/62442katieb/IDConn/blob/main/CODE_OF_CONDUCT.md) during any interaction with the project. | ||
That includes- but is not limited to- online conversations, in-person workshops or development sprints, and when giving talks about the software. | ||
|
||
As stated in the code, severe or repeated violations by community members may result in exclusion from collective decision-making and rejection of future contributions to the ``IDConn`` project. | ||
|
||
## Asking questions about using IDConn | ||
|
||
Please direct usage-related questions to [NeuroStars][link_neurostars], with [the "Software Support" category and the "IDConn" tag][link_neurostars_IDConn]. | ||
The ``IDConn`` developers follow NeuroStars, and will be able to answer your question there. | ||
|
||
## Labels | ||
|
||
The current list of labels are [here][link_labels] and include: | ||
|
||
* [![Good First Issue](https://img.shields.io/badge/-good%20first%20issue-7057ff.svg)](https://github.com/62442katieb/IDConn/labels/good%20first%20issue) | ||
*These issues contain a task that a member of the team has determined should require minimal knowledge of the existing codebase, and should be good for people new to the project.* | ||
If you are interested in contributing to IDConn, but aren't sure where to start, we encourage you to take a look at these issues in particular. | ||
|
||
* [![Help Wanted](https://img.shields.io/badge/-help%20wanted-33aa3f.svg)](https://github.com/62442katieb/IDConn/labels/help%20wanted) | ||
*These issues contain a task that a member of the team has determined we need additional help with.* | ||
If you feel that you can contribute to one of these issues, we especially encourage you to do so! | ||
|
||
* [![Bug](https://img.shields.io/badge/-bug-ee0701.svg)](https://github.com/62442katieb/IDConn/labels/bug) | ||
*These issues point to problems in the project.* | ||
If you find new a bug, please give as much detail as possible in your issue, including steps to recreate the error. | ||
If you experience the same bug as one already listed, please add any additional information that you have as a comment. | ||
|
||
* [![Enhancement](https://img.shields.io/badge/-enhancement-84b6eb.svg)](https://github.com/62442katieb/IDConn/labels/enhancement) | ||
*These issues are asking for new features to be added to the project.* | ||
Please try to make sure that your requested feature is distinct from any others that have already been requested or implemented. | ||
If you find one that's similar but there are subtle differences please reference the other request in your issue. | ||
|
||
## Making a change | ||
|
||
We appreciate all contributions to IDConn, but those accepted fastest will follow a workflow similar to the following: | ||
|
||
**1. Comment on an existing issue or open a new issue referencing your addition.** | ||
|
||
This allows other members of the IDConn development team to confirm that you aren't overlapping with work that's currently underway and that everyone is on the same page with the goal of the work you're going to carry out. | ||
|
||
[This blog][link_pushpullblog] is a nice explanation of why putting this work in up front is so useful to everyone involved. | ||
|
||
**2. Fork IDConn.** | ||
|
||
[Fork][link_fork] the [IDConn repository][link_idconn] to your profile. | ||
|
||
This is now your own unique copy of IDConn. | ||
Changes here won't effect anyone else's work, so it's a safe space to explore edits to the code! | ||
|
||
Make sure to [keep your fork up to date][link_updateupstreamwiki] with the main repository. | ||
|
||
**3. Make the changes you've discussed.** | ||
|
||
Try to keep the changes focused. We've found that working on a [new branch][link_branches] makes it easier to keep your changes targeted. | ||
|
||
When you're creating your pull request, please do your best to follow IDConn's preferred style conventions. | ||
Namely, documentation should follow the [numpydoc](https://numpydoc.readthedocs.io/en/latest/) convention and code should adhere to [PEP8](https://www.python.org/dev/peps/pep-0008/) as much as possible. | ||
|
||
**4. Submit a pull request.** | ||
|
||
Submit a [pull request][link_pullrequest]. | ||
|
||
A member of the development team will review your changes to confirm that they can be merged into the main codebase. | ||
|
||
Please use a sentence-case title for the pull request, and do not include any prefixes (e.g., ``[ENH]``), as we now use labels to distinguish pull request types. | ||
The title should summarize the changes proposed in the pull request, with an emphasis on readability, as pull request titles are used directly in our release notes. | ||
|
||
## Recognizing contributions | ||
|
||
We welcome and recognize all contributions from documentation to testing to code development. | ||
You can see a list of current contributors in our [zenodo][link_zenodo] file. | ||
If you are new to the project, don't forget to add your name and affiliation there! | ||
|
||
## Thank you! | ||
|
||
You're awesome. | ||
|
||
.. note:: | ||
These guidelines are based on contributing guidelines from the [STEMMRoleModels][link_stemmrolemodels] project. | ||
|
||
[link_github]: https://github.com/ | ||
[link_idconn]: https://github.com/62442katieb/IDConn | ||
[link_signupinstructions]: https://help.github.com/articles/signing-up-for-a-new-github-account | ||
[link_react]: https://github.com/blog/2119-add-reactions-to-pull-requests-issues-and-comments | ||
[link_issues]: https://github.com/62442katieb/IDConn/issues | ||
[link_labels]: https://github.com/62442katieb/IDConn/labels | ||
[link_discussingissues]: https://help.github.com/articles/discussing-projects-in-issues-and-pull-requests | ||
[link_neurostars]: https://neurostars.org | ||
|
||
|
||
[link_pullrequest]: https://help.github.com/articles/creating-a-pull-request/ | ||
[link_fork]: https://help.github.com/articles/fork-a-repo/ | ||
[link_pushpullblog]: https://www.igvita.com/2011/12/19/dont-push-your-pull-requests/ | ||
[link_branches]: https://help.github.com/articles/creating-and-deleting-branches-within-your-repository/ | ||
[link_updateupstreamwiki]: https://help.github.com/articles/syncing-a-fork/ | ||
[link_stemmrolemodels]: https://github.com/KirstieJane/STEMMRoleModels | ||
[link_zenodo]: https://github.com/62442katieb/IDConn/blob/main/.zenodo.json |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.