Skip to content
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

added cross-provider unique 'seatet_id' #9

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

tomsail
Copy link
Collaborator

@tomsail tomsail commented Oct 17, 2023

Added a new field seaset_id that contains a unique identifier for the whole catalog.

The 5 providers in the catalog:

  • IOC
  • COOPS
  • EMODNET
  • CMEMS
  • GESLA3
    all have a corresponding field: <provider>_id that is = None if the station has no data from that provider.

There are 2 version now of the catalog:

  1. catalog_full.csv: containing 6115 stations
  2. catalog_minimal.csv that condenses all the stations based on their location >> reduced to 3670 stations.

@pmav99
Copy link
Member

pmav99 commented Oct 30, 2023

the PROXY env variables are BDAP specific and they need to be removed.

@tomsail tomsail marked this pull request as ready for review November 3, 2023 10:18
@tomsail
Copy link
Collaborator Author

tomsail commented Feb 15, 2024

@pmav99 have a look and let me know what you think!

@tomsail
Copy link
Collaborator Author

tomsail commented Mar 6, 2024

nos_id i.e. the station_id parameter for COOPS was not in the catalog.
I had to add it in the Notebook (at the beginning, 4th cell)

Now going through every row of the catalog, I can use searvey directly to fetch stations from both COOPS and IOC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants