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

FireRisk HuggingFace Port #2000

Merged
merged 1 commit into from
Apr 17, 2024
Merged

FireRisk HuggingFace Port #2000

merged 1 commit into from
Apr 17, 2024

Conversation

isaaccorley
Copy link
Collaborator

@isaaccorley isaaccorley commented Apr 15, 2024 •
edited by adamjstewart
Loading

The FireRisk dataset has been rehosted to HuggingFace. This PR updates the download url

Fixes #1996

@isaaccorley isaaccorley added this to the 0.5.3 milestone Apr 15, 2024
@isaaccorley isaaccorley self-assigned this Apr 15, 2024
@github-actions github-actions bot added the datasets Geospatial or benchmark datasets label Apr 15, 2024
@isaaccorley isaaccorley mentioned this pull request Apr 15, 2024
Closed
@adamjstewart
Copy link
Collaborator

My only concern is CharmonyShen/FireRisk#2

@adamjstewart
Copy link
Collaborator

Also, if it is CC BY-NC 4.0, you need to attribute the source and describe any changes in the HF README: https://creativecommons.org/licenses/by-nc/4.0/deed.en

@isaaccorley
Copy link
Collaborator Author

Also, if it is CC BY-NC 4.0, you need to attribute the source and describe any changes in the HF README: https://creativecommons.org/licenses/by-nc/4.0/deed.en

Thanks for pointing that out. I'll update the readme.

@adamjstewart
Copy link
Collaborator

#2000, nice 😎

@isaaccorley
Copy link
Collaborator Author

@adamjstewart I've updated the README on HuggingFace

adamjstewart
adamjstewart approved these changes Apr 17, 2024
View reviewed changes
Copy link
Collaborator

@adamjstewart adamjstewart left a comment

Choose a reason for hiding this comment

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

The license is still unclear, but given that the author is too busy to clarify, they are hopefully also too busy to complain that we are rehosting.

@adamjstewart adamjstewart merged commit 8b24ca1 into microsoft:main Apr 17, 2024
15 checks passed
@adamjstewart adamjstewart modified the milestones: 0.5.3, 0.6.0 Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adamjstewart adamjstewart adamjstewart approved these changes

Assignees

@isaaccorley isaaccorley

Labels
datasets Geospatial or benchmark datasets
Projects
None yet
Milestone
0.6.0
Development

Successfully merging this pull request may close these issues.

Auto download fails for FireRisk
2 participants
@isaaccorley @adamjstewart