Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Climatology for all SA Moorings #3

Open
ribeiron opened this issue Aug 8, 2023 · 4 comments
Open

Climatology for all SA Moorings #3

ribeiron opened this issue Aug 8, 2023 · 4 comments
Assignees

Comments

@ribeiron
Copy link
Collaborator

ribeiron commented Aug 8, 2023

Here there is a code that does that for the SA NRS station mooring: https://github.com/oceanhackweek/ohw23_proj_SAupwelling/blob/main/Notebooks/ExampleCodeFromOHW22.ipynb

@vinisalazar
Copy link
Collaborator

Hi @ribeiron, I can try going through the existing code and separating stuff into functions so we can iterate over all moorings more effectively, would that be useful?

@ribeiron
Copy link
Collaborator Author

ribeiron commented Aug 8, 2023

Sounds great! Do you mind creating a new code in this repo instead of updating directly on the file of the old one? Just so I can keep the code I already understand?

@vinisalazar
Copy link
Collaborator

@ribeiron I don't have push access to this repo, could you add me as a collaborator? Or should I fork it + open PRs for each change?

Also, is it okay to commit directly to main? Or should we create separate branches?

Thanks

@ribeiron
Copy link
Collaborator Author

ribeiron commented Aug 8, 2023

I think it is fixed. I replied to this on the Slack Channel, but I reckon we can commit to main to start with and then reassess.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants