-
Notifications
You must be signed in to change notification settings - Fork 2
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 #30 from MetOffice/revert-29-develop
Revert "Merge v1.0 Develop into Master"
- Loading branch information
Showing
30 changed files
with
6,024 additions
and
5,108 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,2 @@ | ||
.ipynb_checkpoints/ | ||
notebooks/.ipynb_checkpoints/ | ||
/data/stash_select | ||
*.pyc |
This file was deleted.
Oops, something went wrong.
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 |
---|---|---|
@@ -1,70 +1,29 @@ | ||
# Contributing Guidelines | ||
|
||
You'd like to help? Great! :tada: | ||
|
||
[Clone your own local copy](https://help.github.com/en/articles/cloning-a-repository) of this repositry run the following in your terminal: | ||
|
||
```shell | ||
git clone [email protected]:MetOffice/PyPRECIS.git | ||
``` | ||
|
||
Consider [creating a conda environment](https://docs.conda.io/projects/conda/en/latest/user-guide/tasks/manage-environments.html) from the PyPRECIS dependencies specified in the `environment.yml` file: | ||
```shell | ||
conda env create -f environment.yml | ||
``` | ||
Remember to activate your new environment: | ||
```shell | ||
conda activate pyprecis-environment | ||
``` | ||
|
||
:exclamation: *Note: As of v1.0 we are unable to provison the model data necessary for reproducing the full PyPRECIS learning environment via github due to it's substanation size. Contact the PRECIS team for more information.* | ||
|
||
## Before you start... | ||
Read through the current issues to see what you can help with. If you have your own ideas for improvements, please start a new issues so we can track and discuss your improvement. You must create a new branch for any changes you make. | ||
|
||
**Please take note of the following guidelines when contributing to the PyPRECIS repository.** | ||
|
||
* Please do **not** make changes to the `master` branch. The `master` branch is reserved for files and code that has been fully tested and reviewed. Only the core PyPRECIS developers can/should push to the `master` branch. | ||
Please take note of the following guidelines when contributing to the PyPRECIS repository: | ||
|
||
* Please do **not** make changes to the `master` branch. The `master` branch is reserved for files and code that has been fully tested and reviewed. If you think something is ready to be pushed to the `master` branch please check with Saeed or Hamish first. | ||
* The `develop` branch contains the latest holistic version of the `PyPRECIS` repository. Please branch off `develop` to fix a particular issue or add a new feature. | ||
* Please use the following tokens at the start of a new branch name to help sign-post and group branches: | ||
|
||
Name | Description | ||
---- | ----------- | ||
new | Branch adding new code/files that don't exist in the repo | ||
fix | Branch modifying code/files that already exist in the repo. | ||
junk | Throwaway branch created to experiment | ||
|
||
* Git can pattern match branches to to give you an overview of all (e.g. fix) branches: | ||
```shell | ||
``` | ||
new Branch adding new code/files that don't exist in the repo. | ||
fix Branch modifying code/files that already exist in the repo. | ||
junk Throwaway branch created to experiment | ||
``` | ||
Git can pattern match branches to to give you an overview of all (e.g. fix) branches: | ||
``` | ||
git branch --list "fix/*" | ||
``` | ||
* Use a forward slash to separate the token from the branch name. For example: | ||
``` | ||
new/Wks10 | ||
fix/Wks2_units | ||
new/Worksheet5 | ||
fix/worksheet2aUnits | ||
``` | ||
* When you think your branch is ready to be merged into `develop`, open a new pull request. | ||
|
||
## Signposting | ||
* **Issues** are tracked and discussed under the Issues tab. Please use issues to disucss proposed changes or capture improvements needed to work towards the next milestone. Issues or improvements that contribute to the next milestone to be captured in thr Wiki tab. | ||
* **Pull requests** show branches that are currently under review. New pull requests are created in reponse to branch fixes identified and recorded in the Issues tab. | ||
* **Wiki** is used for summarising update aims for future versions of the notebooks, and to record speculative improvements that cannot be action in the current milestone. | ||
|
||
|
||
|
||
Other more general points to note: | ||
|
||
* **Avoid long descriptive names.** Long branch names can be very helpful when you are looking at a list of branches but it can get in the way when looking at decorated one-line logs as the branch names can eat up most of the single line and abbreviate the visible part of the log. | ||
* **Avoid long descriptive names.** Long branch names can be very helpful when you are looking at a list of branches. But it can get in the way when looking at decorated one-line logs as the branch names can eat up most of the single line and abbreviate the visible part of the log. | ||
* **Do not use bare numbers.** Do not use use bare numbers (or hex numbers) as part of your branch naming scheme. | ||
|
||
## CONTRIBUTING.ipynb | ||
The `CONTRIBUTING.ipyn` file contains the worksheet style guide. Please consult this for information on formatting the new and ammended worksheets in a consistent style. | ||
|
||
**If in doubt, please contact the PRECIS team ([email protected]) if you | ||
have questions.** | ||
|
||
<h5 align="center"> | ||
<img src="notebooks/img/MO_MASTER_black_mono_for_light_backg_RBG.png" width="200" alt="Met Office"> <br> | ||
© British Crown Copyright 2018 - 2019, Met Office | ||
</h5> | ||
**If in doubt, please contact Saeed Sadri ([email protected]) or Hamish Steptoe ([email protected]) if you | ||
have questions** |
Oops, something went wrong.