-
Notifications
You must be signed in to change notification settings - Fork 32
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
2022-09-19-mp-elemental-reference-entries.json missing #17
Comments
Are you sure you're running the latest code? It should prompt you to download that file on the fly if it doesn't exist yet. Maybe this matbench-discovery/matbench_discovery/data.py Lines 204 to 221 in 3085c90
|
The isatty() check is ok. The error is one line below: matbench-discovery/matbench_discovery/data.py Line 217 in 0da443f
Instead of
You should use
With this fix, the query appears and when answering yes the file is downloaded and cached. |
Oops, that was committed by accident. I'd briefly considered trying to support |
* fix load_train_test() for compressed figshare data (closes #20) * load_train_test() only accept answer 'y' or 'n' (as orig intended) (close #17) * add test covering load_train_test() with compressed JSON file from URL * mv run-scripts.yml test-scripts.yml * add slow-tests.yml for running slow tests only on PR merges (to save CI budget)
* fix load_train_test() for compressed figshare data (closes #20) * load_train_test() only accept answer 'y' or 'n' (as orig intended) (close #17) * add test covering load_train_test() with compressed JSON file from URL * mv run-scripts.yml test-scripts.yml * add slow-tests.yml for running slow tests only on PR merges (to save CI budget)
* fix load_train_test() for compressed figshare data (closes #20) * load_train_test() only accept answer 'y' or 'n' (as orig intended) (close #17) * add test covering load_train_test() with compressed JSON file from URL * mv run-scripts.yml test-scripts.yml * add slow-tests.yml for running slow tests only on PR merges (to save CI budget)
Solved this issue using:
The text was updated successfully, but these errors were encountered: