Closed kavanase closed 3 months ago
The changes in setup.py
involve updating the paths for package data related to the maml
package. Specifically, the paths for MegNet model files have been modified to ensure they point to the correct directories. This adjustment enhances the accuracy and functionality of the package's data management.
File | Change Summary |
---|---|
setup.py |
Updated package data paths for maml package, changing paths for MegNet model files from "describers/data/megnet_models/*.json" to "describers/data/megnet_models/*" and from "describers/data/megnet_mdoels/*.hdf5" to "describers/data/m3gnet_models/matbench_mp_e_form/0/m3gnet/*" . |
setup.py (1)
`125-126`: The changes to `package_data` look correct and should resolve the issue with missing model data for `M3GNetStructure`. Please ensure to verify that these files are correctly included in the built package.
Thanks
When trying to parse
Structure
objects withM3GNetStructure
, the following error is encountered:which seems to be because the M3GNet model data from
maml/describers/data/m3gnet_models/matbench_mp_e_form/0/m3gnet
is not available in the package data – so works fine for GitHub Actions tests run in the cloned repo, but not when actually used elsewhere in scripts/notebooks. Fixingpackage_data
insetup.py
fixes this.Also fixes a typo in
setup.py
formegnet_models
(previouslymegnet_mdoels
).I would also note that this also fails with recent
tensorflow
versions (2.16
) which useskeras
v3, giving the following error:while downgrading to
tensorflow==2.15
andkeras<3
fixes this issue. I'm not sure if you want to update the requirements for this, or update the code to work with recent versions.When fixing all these issues, the code runs, but also throws many warnings: