search

materialsproject / emmet

Be a master builder of databases of material properties. Avoid the Kragle.
https://materialsproject.github.io/emmet/
Other
49 stars 63 forks source link

make ml dependencies optional for emmet-builders #1022

Closed tsmathis closed 1 month ago

tsmathis commented 1 month ago

The cuda toolchain installed with emmet-core[ml] is bloating the dependencies of emmet-builders currently. The ML builder needs a closer look for how we are going to run it on a regular basis, so it doesn't make sense to drag along all the dependencies at the moment

codecov-commenter commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.03%. Comparing base (51bcc82) to head (993857d).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1022 +/- ## ======================================= Coverage 90.03% 90.03% ======================================= Files 142 142 Lines 13395 13395 ======================================= Hits 12060 12060 Misses 1335 1335 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.