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.

Sign up for GitHub

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

Jump to bottom

Test data with pooch #89

Closed
MaceKuailv opened this issue Oct 18, 2023 · 3 comments
Closed

Test data with pooch #89

MaceKuailv opened this issue Oct 18, 2023 · 3 comments
Labels
dependencies Pull requests that update a dependency file

Comments

@MaceKuailv
Copy link
Owner

A version of the following code is used to configure the tests

import pooch
pooch_testdata = pooch.create(
    path=pooch.os_cache("seaduck"),
    base_url="doi:10.5281/zenodo.7949168",
    registry=None,
)
pooch_testdata.load_registry_from_doi()  # Automatically populate the registry

This result in a "KeyError: "Key"".

Apparently, folks are already working on this and has already opened a pull request fatiando/pooch#371.
@MaceKuailv MaceKuailv added the dependencies Pull requests that update a dependency file label Oct 18, 2023
@MaceKuailv MaceKuailv mentioned this issue Oct 18, 2023
Merged
@MaceKuailv
Copy link
Owner Author

OMS-NetZero/FAIR#138

I know so little about pooch, but this seems like a valid way of getting around the doi issue.

@MaceKuailv
Copy link
Owner Author

pooch v1.8.0 have this issue resolved. However, the CI process is still downloading some older version at this moment. I will wait a few more days.

@MaceKuailv
Copy link
Owner Author

closed after github CI updated to pooch 1.8.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@MaceKuailv