search

oceanhackweek / ohw23_proj_SAupwelling

Apache License 2.0
2 stars 1 forks source link

Climatology for all SA Moorings #3

Open ribeiron opened 1 year ago

ribeiron commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

@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 commented 1 year ago

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.