Skip to content

Adds GLORYS BGC#352

Open
jagoosw wants to merge 7 commits into
mainfrom
jsw/glorys-bgc
Open

Adds GLORYS BGC#352
jagoosw wants to merge 7 commits into
mainfrom
jsw/glorys-bgc

Conversation

@jagoosw

@jagoosw jagoosw commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Adds GLORYS hindcast and analysis/forecast BGC datasets

@codecov

codecov Bot commented Jun 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 36 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/DataWrangling/GLORYS/GLORYS_bgc.jl 0.00% 21 Missing ⚠️
src/DataWrangling/GLORYS/GLORYS.jl 0.00% 13 Missing ⚠️
ext/NumericalEarthCopernicusMarineExt.jl 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@jagoosw jagoosw changed the title Add GLORYS BGC Adds GLORYS BGC Jun 18, 2026
@navidcy navidcy added the data wrangling 🗃️ JRA55, ECCO, ERA5, and friends label Jun 20, 2026
@jagoosw jagoosw marked this pull request as ready for review June 23, 2026 22:02
Comment thread src/DataWrangling/GLORYS/GLORYS_bgc.jl Outdated
copernicusmarine_dataset_id(::GLORYSBGCDaily) = "cmems_mod_glo_bgc_my_0.25deg_P1D-m"
copernicusmarine_dataset_id(::GLORYSBGCMonthly) = "cmems_mod_glo_bgc_my_0.25deg_P1M-m"

available_variables(::GLORYSBGCDaily) =

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you align these =>?

:phosphate => "po4",
:silicate => "si")

available_variables(::GLORYSBGCMonthly) =

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also here and below?

Comment thread src/DataWrangling/GLORYS/GLORYS_bgc.jl Outdated
Comment thread src/DataWrangling/GLORYS/GLORYS_bgc.jl Outdated
Comment thread src/DataWrangling/GLORYS/GLORYS_bgc.jl Outdated
Comment thread src/DataWrangling/GLORYS/GLORYS_bgc.jl Outdated
jagoosw and others added 2 commits June 24, 2026 16:34
Co-authored-by: Simone Silvestri <silvestri.simone0@gmail.com>
@jagoosw

jagoosw commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator Author

Both of these CI failures are unrelated to this PR so is it fine for me to merge anyway?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data wrangling 🗃️ JRA55, ECCO, ERA5, and friends

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants