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

Bugfix zenodo api change #80

Closed
wants to merge 18 commits into from
Closed

Bugfix zenodo api change #80

wants to merge 18 commits into from

Conversation

dfalster
Copy link
Member

As documented in #79 , the Zenodo API seems to have changed, breaking our download feature.

This commit updates the internals to work with the latest changes.

Specifically:

  • the way to access json for all versions has changed (changed url structure, and for id we now use one of the record ids, rather than the conceptid)
  • the call to download file has changed
  • format of the API json has changed

Also

  • added record id to the table of versions
  • put a check in to remove "v" from any version entered by user

fontikar and others added 18 commits January 12, 2023 16:04
…king for new version, waiting for austraits.build update
…rough the internal data subsets for version 3.0.2 and version 4.0.0. Pivot_ needs austraits.build fixes to implenment. Some minor fix to pivot_wider for dependencies #60
Some packages needed to make plots are included in suggests. This means that core functions of package may not work.
- reuse outputs from previous function calls to reduce runtime
- reduce dataset sizes for slow functions (summarise_trait_means, trait_pivot_wider, plot_locations)
- silence some outputs

closes #62
@dfalster
Copy link
Member Author

Wrong branch

@dfalster dfalster closed this Oct 16, 2023
@dfalster
Copy link
Member Author

replace by #81

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