Skip to content

Bug for ECCO2 dataset metadata #470

New issue
New issue
Open
#479
Open
Bug for ECCO2 dataset metadata#470
#479
Labels
bugSomething isn't workingdata wranglingWe must feed the models so they don't get crankytestsHelpful for getting a good night's sleep
@navidcy

Description

@navidcy
navidcy
opened on Apr 15, 2025

Definitely a bug here

ClimaOcean.jl/src/DataWrangling/ECCO/ECCO_metadata.jl

Lines 133 to 135 in b4f2e56

# URLs for the ECCO datasets specific to each dataset
metadata_url(m::Metadata{<:ECCO2Daily}) = ECCO2_url * "monthly/" * short_name(m) * "/" * metadata_filename(m)
metadata_url(m::Metadata{<:ECCO2Monthly}) = ECCO2_url * "daily/" * short_name(m) * "/" * metadata_filename(m)

I can fix it but it also points to the need to include the ECCO2 datasets in the test_downloading.jl and/or test_ecco.jl.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingdata wranglingWe must feed the models so they don't get crankytestsHelpful for getting a good night's sleep

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions