Skip to content

Update fetch_zenodo_predictors.r#20

Merged
mmyrte merged 3 commits intomainfrom
update_zenodo
Apr 27, 2026
Merged

Update fetch_zenodo_predictors.r#20
mmyrte merged 3 commits intomainfrom
update_zenodo

Conversation

@ManuelKurmann
Copy link
Copy Markdown
Contributor

Update fetch_zenodo_predictors() for new Zenodo record

Update fetch_zenodo_predictors() for new Zenodo record
@ManuelKurmann ManuelKurmann requested a review from mmyrte April 9, 2026 12:58
@mmyrte mmyrte mentioned this pull request Apr 13, 2026
2 tasks
@mmyrte mmyrte linked an issue Apr 13, 2026 that may be closed by this pull request
2 tasks
@mmyrte
Copy link
Copy Markdown
Member

mmyrte commented Apr 13, 2026

fetch_zenodo_predictors(url = "file:///some/path/evoland-ch-data.zip")

There's a usage of fetch_zenodo_predictors here that fell back to a local file, you could remove this - while you're touching the file, you could mention something about the download size so users have an expectation for how large it is/download time.

Update Zenodo download example in vignette
@ManuelKurmann
Copy link
Copy Markdown
Contributor Author

ManuelKurmann commented Apr 15, 2026

fetch_zenodo_predictors(url = "file:///some/path/evoland-ch-data.zip")

There's a usage of fetch_zenodo_predictors here that fell back to a local file, you could remove this - while you're touching the file, you could mention something about the download size so users have an expectation for how large it is/download time.

fetch_zenodo_predictors(url = "file:///some/path/evoland-ch-data.zip")

There's a usage of fetch_zenodo_predictors here that fell back to a local file, you could remove this - while you're touching the file, you could mention something about the download size so users have an expectation for how large it is/download time.

Done. Are my changes what you ment?

Copy link
Copy Markdown
Member

@mmyrte mmyrte left a comment

Choose a reason for hiding this comment

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

All good, please merge :)

@mmyrte mmyrte merged commit 8c7b75f into main Apr 27, 2026
@mmyrte mmyrte deleted the update_zenodo branch April 27, 2026 07:22
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.

reupload raw data

2 participants