-
Notifications
You must be signed in to change notification settings - Fork 17
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
Comparison to CO2SYS #6
Comments
Towards #6. Still need to do actual comparison.
@oscarbranson Thanks for putting the effort into this project. It looks great. I could potentially help with the comparison to CO2SYS (or one of the other issues on the TODO list). I am a physical oceanographer, and have recently transferred most of my work from Matlab to Python. |
Thanks @tompc35 - that would be really useful. I've actually made a start on this, but would welcome help. I was thinking of comparing cbsyst to GLODAP/WOA data, after Orr er al (2015). I'd like the comparison to be self-contained, so any user can run it on their own system - i.e. get the data, run it through CO2SYS.m, and generate a 'comparison report' containing plots comparing cbsyst to CO2SYS.m (e.g. scripts making the GLODAP plots in the readme). This could be wrapped in two scripts, and the workflow could be something like:
So far, I've written get_GLODAP_WOA_data.py, which scrapes and combines the GLODAPv1/WOA data, and I've run this through CO2SYS.m so have a comparison dataset offline. plot_CO2SYS_comparison.py is where work is needed. All that said, it might be worth holding off a couple of weeks until #13 and #8 are closed, which will add the final features to to match the full capabilities of CO2SYS.m. So, I think a roadmap to getting this is done is something like:
I've created a branch to work on this called co2sys_compare. Feel free to contribute! Happy to discuss further on here. |
After Orr et al (2015).
The text was updated successfully, but these errors were encountered: