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

temporarily disable OpenAltimetry API tests #459

Merged
merged 3 commits into from
Oct 18, 2023
Merged

temporarily disable OpenAltimetry API tests #459

merged 3 commits into from
Oct 18, 2023

Conversation

JessicaS11
Copy link
Member

As noted in #454, the OpenAltimetry programmatic API is currently unavailable during the transition of OA hosting from UCSD (Univ CA San Diego) to NSIDC (NASA DAAC). A ticket marked critical was issued, but the fix is not anticipated for another week or two. In order to allow other development to proceed, this PR disables tests depending on the OA API. The plan is to reinstate them as soon as the API becomes available again.

@github-actions
Copy link

Binder 👈 Launch a binder notebook on this branch for commit 845b18c

I will automatically update this comment whenever this PR is modified

@JessicaS11 JessicaS11 requested a review from rwegener2 October 18, 2023 14:14
Copy link
Contributor

@rwegener2 rwegener2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Great idea to put a warning in.

@JessicaS11 JessicaS11 merged commit d03f9fb into development Oct 18, 2023
1 check passed
@JessicaS11 JessicaS11 deleted the fix-oa branch October 18, 2023 17:05
JessicaS11 added a commit that referenced this pull request Nov 15, 2023
* add OA API warning
* comment out tests that use OA API

---------

Co-authored-by: GitHub Action <[email protected]>
JessicaS11 added a commit that referenced this pull request Jan 5, 2024
* add OA API warning
* comment out tests that use OA API

---------

Co-authored-by: GitHub Action <[email protected]>
JessicaS11 added a commit that referenced this pull request Jan 5, 2024
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