Skip to content

docs: document OPeNDAP URL retrieval (#287) - #1448

Merged
betolink merged 3 commits into
earthaccess-dev:mainfrom
svsam:issue-287-opendap-links
Sep 1, 2026
Merged

betolink merged 3 commits into
earthaccess-dev:mainfrom
svsam:issue-287-opendap-links

Conversation

@svsam

@svsam svsam commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Description

Closes #287.

This PR documents how to retrieve OPeNDAP service URLs from granule search results. The example filters each granule's umm.RelatedUrls entries using:

  • Type == "USE SERVICE API"
  • Subtype == "OPENDAP DATA"

It uses defensive .get() access and explains that collections without OPeNDAP metadata return an empty list.

This is a documentation-only change so no public APIs, library code, tests, or fixture files were changed.

Test data

Existing fixture Granules OPeNDAP URLs retrieved Ordinary data URLs excluded
GPM 1 1 1
LAADS 1 1 1
Daymet 10 10 20

Verification

  • pytest tests/unit/test_results.py -q -p no:cacheprovider — 16 passed
  • mkdocs build --no-strict — passed
  • git diff --check — passed
  • Confirmed that only docs/user/howto/access-data.md changed

"Ready for review" checklist

  • Place this Pull Request (PR) in draft until it is ready for review (see below)
  • Please review our Pull Request Guide
  • Mark "ready for review" after following instructions in the guide

Merge checklist

  • PR title is descriptive
  • PR body contains links to related and resolved issues (e.g. closes #1)
  • If needed, CHANGELOG.md updated
  • If needed, docs and/or README.md updated
  • If needed, unit tests added (documentation-only change; existing tests pass)
  • All checks passing (pending GitHub CI)
  • At least one approval

Need help? We welcome contributions at every experience level. You don't have to
write tests alone — open your PR and ask for help. It's also fine to let GitHub run tests
for you, via Continuous Integration (CI),
instead of running them locally. If anything fails and you're not sure why, just
mention @earthaccess-dev/maintainers in a comment and we'll work with you!


📚 Documentation preview 📚: https://earthaccess--1448.org.readthedocs.build/en/1448/

@github-actions

github-actions Bot commented Aug 28, 2026

Copy link
Copy Markdown

Binder 👈 Launch a binder notebook on this branch for commit 4f2c912

I will automatically update this comment whenever this PR is modified

Binder 👈 Launch a binder notebook on this branch for commit 85a0bb4

Binder 👈 Launch a binder notebook on this branch for commit 1d455c1

@svsam
svsam marked this pull request as ready for review August 29, 2026 13:31
@betolink
betolink requested a lite review from Copilot August 31, 2026 19:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@betolink
betolink self-requested a review August 31, 2026 19:43

@betolink betolink left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for this contribution @svsam ! I think it's very useful to have documentation on how to collect OPeNDAP endpoints. Approving the PR

@betolink
betolink merged commit 1902cd5 into earthaccess-dev:main Sep 1, 2026
7 of 8 checks passed
@betolink betolink mentioned this pull request Sep 3, 2026
11 tasks
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.

retrieve OPenDAP urls

3 participants