Closed njzjz closed 2 weeks ago
pyproject.toml (3)
`50-50`: Moving `mendeleev` from `dev-dependencies` to `dependencies` ensures it is always available, preventing runtime errors. --- `50-50`: Removal of `mendeleev` from `dev-dependencies` is consistent with its addition to `dependencies`. --- `50-50`: Update to `sphinx>=3.1.1` in the docs section is approved. Please verify compatibility with existing documentation setups.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 82.70%. Comparing base (
c644314
) to head (a41dcb6
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
shall we add dpdata and mendeleev to dependencies?
I don't find the meaning of using dpdata - the element table has already been hard-coded in econf_embd.py
yes, please remove the import of dpdata.
Fix #3743.
Summary by CodeRabbit
dpdata
andmendeleev
.pyproject.toml
to includemendeleev
in the main section and update packages for testing and documentation.