cioos-siooc / cioos-yaml-to-erddap

Converts CIOOS yaml metadata to erddap
0 stars 0 forks source link

add more erddap global attributes #2

Open n-a-t-e opened 3 years ago

n-a-t-e commented 3 years ago

Between this PR and my previous commit to master, I think I've covered all the fields suggested by @JessyBarrette in #1

Fixes #1

JessyBarrette commented 3 years ago

I'm not sure if I see the contacts added in here. Can you confirm that the different contacts are added if available:

JessyBarrette commented 1 year ago

@n-a-t-e I refactor a litle the code to make it easier to follow the different inputs, I also regrouped all the contact-specific attributes and integrated more contact fields related to ioos 1.2

The following list of things were just added since:

  1. add more to publisher, creator and contributor fields
  2. creator maps to first contact owner
  3. institution match creator organization
  4. publisher maps to first contact distributor
  5. id refers always to to the cioos identifier
  6. naming_authority = ca.cioos
  7. metadata_link and infoUrl refers to ckan National record page
  8. geospatial_* attributes were dropped since ERDDAP populates them based on the data

A few mappings are missing to be compliant to IOOS 1.2. I'll generate a new issue to bring up to metadata TT to discuss that.

I also added Pytest in their so that we can add more tests in a future release to test different issues encountered.

I also decided to keep the alternative language instead of fixing one language to the erddap metadata. This is maybe for a broader discussion.