search

dianna-ai / dianna

Deep Insight And Neural Network Analysis
https://dianna.readthedocs.io
Apache License 2.0
44 stars 13 forks source link

Automatic data downloading with pooch #796

Closed loostrum closed 2 weeks ago

loostrum commented 3 weeks ago

A downloader utility was added, which contains an overview of all data files we use, hosted on either Github or Zenodo. With the dianna.utils.downloader.download function, one can download and use any of these files. The files are downloaded to an OS cache directory by default, and the checksums are verified so any changes in the online files are detected.

The tutorials were updated to use this functionality.

While rerunning the notebooks I noticed the colormaps were mostly wrong (default viridis instead of bwr). This is due to https://github.com/dianna-ai/dianna/pull/776, where the colormaps were removed from the notebooks, but the defaults were not set in the dianna code for all modalities (only in the dashboard it seems). In this PR I added the bwr default to the visualization module where not already present.

review-notebook-app[bot] commented 2 weeks ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

elboyran commented 2 weeks ago

@loostrum if all models are downloaded from Zenood now, why to keep the models folder? Or at least to remove the non-needed files?