-
Notifications
You must be signed in to change notification settings - Fork 37
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
New breaking argovis API version 2 #246
Comments
And because the old API is not turned off, our method to check if argovis is alive was misleading: |
Dear Guillaume,Thanks for your message. When you have a moment, could you please point Argopy to the legacy version of Argovis (https://argovisbeta02.colorado.edu/) as described in the email I sent you and Kevin earlier? As described in the message, we will be in touch again about a transition to the new version of Argovis.Thanks,DonataDonata Giglio (She/Her)Assistant ProfessorDepartment of Atmospheric and Oceanic SciencesUniversity of Colorado BoulderMail: 311 UCB | Boulder, CO 80309-0311 Phone: +1-303-735-4354Office: SEEC N253HomepageOn Feb 15, 2023, at 4:42 AM, Guillaume Maze ***@***.***> wrote:
I've noticed this morning that argovis has moved to API version 2
and it's not backward compatible, hence it's breaking argopy ds='argovis' data source
and it now requires to register and get an API key
That's a lot to handle to keep it working with argopy
any will from your team @donatagiglio to make a PR and keep argovis available in argopy ?
https://argovis.colorado.edu/
https://argovis-api.colorado.edu/docs/
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
hi @donatagiglio |
for the future PR, the way to pass a key in the request header using our filesystem would be something like: client_kwargs = {'headers': {'x-argokey': apikey}}
fs = argopy.stores.httpstore(client_kwargs=client_kwargs)
fs.open_json('https://argovis-api.colorado.edu/argo?platform=6902915&data=all') |
This issue was marked as staled automatically because it has not seen any activity in 90 days |
I've noticed this morning that argovis has moved to API version 2
and it's not backward compatible, hence it's breaking argopy ds='argovis' data source
and it now requires to register and get an API key
That's a lot to handle to keep it working with argopy
any will from your team @donatagiglio to make a PR and keep argovis available in argopy ?
https://argovis.colorado.edu/
https://argovis-api.colorado.edu/docs/
Example of request for a float:
The text was updated successfully, but these errors were encountered: