ioos / erddap-gold-standard

Contains the 'gold standard' ERDDAP configuration, with datasets compliant with IOOS Metadata Profile 1.2
https://standards.sensors.ioos.us/erddap/index.html
8 stars 17 forks source link

Use nccsv for example datasets for simplicity/visbility #51

Closed srstsavage closed 5 months ago

srstsavage commented 1 year ago

Using the text-based nccsv format allows us to the self describing/embedded metadata features of NetCDF while also allowing easy visibility and browsing of the source data (text is more accessible than binary files).

https://coastwatch.pfeg.noaa.gov/erddap/download/NCCSV.html

MathewBiddle commented 1 year ago

My concern with this is IOOS does link to these files as Gold Standard Example datasets for the IOOS Metadata Profile and I don't think we want to be recommending nccsv as the standard format. https://ioos.github.io/ioos-metadata/ioos-metadata-profile-v1-2.html#gold-standard-example-datasets

IMO we should be exemplifying how to load datasets from a variety of formats. So, I propose that we only change one of these netCDF files to nccsv, but leave the rest as nc. If it would be beneficial to the community, we can provide a .cdl next to the .nc for readability on GH.

srstsavage commented 1 year ago

@MathewBiddle That’s a valid concern. I hate violating DRY but what if we kept both nccsv and nc copies of the example datasets here, made the nc files the default, and included a script to generate the nc files from the nccsv files? That way we could use the nccsv files as the source of truth and cleanly track changes to them via git, but also demonstrate running ERDDAP with standard nc datasets.

MathewBiddle commented 1 year ago

That seems like a viable approach. I'm onboard with it, and interested in what the script would look like.

MathewBiddle commented 11 months ago

@srstsavage FYI

The default branch has been renamed! master is now named main

See https://github.com/ioos/erddap-gold-standard/issues/52

srstsavage commented 5 months ago

Closing in favor of the CDL approach in #70