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

Download files to right locations to allow plotting. #201

Merged
merged 5 commits into from
Feb 6, 2025
Merged

Conversation

CompRhys
Copy link
Collaborator

@CompRhys CompRhys commented Feb 6, 2025

Files were removed from version control but now need a way to download easily to allow plotting scripts and analysis to be used.

Also some clean-up of code decay wrt enums and slight change to dpa LAM model to avoid Alex sAlex duplication in reported training data.

…ing it to be removed from pymatviz given deprecated terminology (see #131)
…e the plots themselves again. blocked as the file urls are private
- add maybe_auto_download_file() used in Model.(discovery|geo_opt|kappa_103)_path
- introduce env var MBD_AUTO_DOWNLOAD_FILES and MBD_CACHE_DIR for download control
- Files enum add abstractmethods for url and label properties so child classes must implement
- test coverage for new download functionality
- remove scripts/download_model_preds_from_figshare.py
Copy link
Owner

@janosh janosh left a comment

Choose a reason for hiding this comment

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

thanks for all the key fixes across scripts! 👍

i didn't have time yet but was planning to implement the same on-demand auto-download functionality for model preds as we already have for DataFiles. now implemented in 9f4120f

@janosh janosh merged commit 3516d23 into main Feb 6, 2025
7 checks passed
@janosh janosh deleted the download-files branch February 6, 2025 12:54
@janosh janosh added data Data loading and processing ux User experience labels Feb 6, 2025
@janosh janosh changed the title [WIP] Download files to right locations to allow plotting. Download files to right locations to allow plotting. Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data Data loading and processing ux User experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants