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

Icebox: Move to new Zenodo API #183

Open
Tracked by #2939
jdangerx opened this issue Oct 16, 2023 · 4 comments
Open
Tracked by #2939

Icebox: Move to new Zenodo API #183

jdangerx opened this issue Oct 16, 2023 · 4 comments

Comments

@jdangerx
Copy link
Member

jdangerx commented Oct 16, 2023

See also catalyst-cooperative/pudl#2939 -

Zenodo migrated to their new API, we need to use it if we want to keep creating new archives.

  • use f"/records/{record_number}/files/{name}" in the datapackage.json
  • hit the correct endpoints in the ZenodoDepositor
  • upgrade to frictionless>=5.0?

Some details on what the "correct endpoints" are: it seems like the basic "make a draft, edit it, publish it" flow is the same as before, which is good.

We have the existing operations defined on our ZenodoDepositor:

Sandbox changes:

  • API values have changed to match regular DOI values with a 5281 prefix, and all sandbox archive DOI values have changed. These will need to be updated.
@zaneselvans
Copy link
Member

Automatically adding the catalyst-cooperative community would be great -- or maybe, add it if the previous version of the record was in the community. Right now the fact that a record is in a community doesn't seem to be inherited by subsequent versions which is very annoying, and leads to the most recent records often not showing up.

@zaneselvans
Copy link
Member

zaneselvans commented Oct 16, 2023

I notice that we do not list frictionless as one of the dependencies in this module, but it does get installed, because we depend on catalystcoop.pudl which depends on catalystcoop.ferc-xbrl-extractor which depends on frictionless.

We also don't use frictionless in this package, even though we redefine (Pydantic) classes which mirror the Resource and DataPackage concepts. This seems like a recipe for ending up with outputs that look like data packages, but may or may not actually conform to the specification.

@jdangerx
Copy link
Member Author

Ooooh yeah. I noticed that dependency funniness but did not connect the dots with "we don't actually use frictionless for data packages." We definitely should update that - could be a separate PR, though.

@e-belfer e-belfer linked a pull request Oct 19, 2023 that will close this issue
27 tasks
@jdangerx jdangerx removed their assignment Oct 31, 2023
@e-belfer
Copy link
Member

e-belfer commented Nov 6, 2023

According to email communications with Zenodo this API will not be launched for another year and it's behavior is expected to change. So let's pause on this.

@e-belfer e-belfer changed the title Move to new Zenodo API Icebox: Move to new Zenodo API Nov 22, 2023
@e-belfer e-belfer assigned jdangerx and unassigned e-belfer Jan 30, 2024
@jdangerx jdangerx removed their assignment May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Icebox
Development

Successfully merging a pull request may close this issue.

3 participants