Skip to content

Commit 380d2f2

Browse files
authored
[Hot Fix] Fix setuptools for pymatgen packaging (materialsproject#3934)
1 parent 0234182 commit 380d2f2

File tree

2 files changed

+4
-7
lines changed

2 files changed

+4
-7
lines changed

pyproject.toml

+4-1
Original file line numberDiff line numberDiff line change
@@ -131,9 +131,12 @@ feff_plot_cross_section = "pymatgen.cli.feff_plot_cross_section:main"
131131
feff_plot_dos = "pymatgen.cli.feff_plot_dos:main"
132132
get_environment = "pymatgen.cli.get_environment:main"
133133

134+
[tool.setuptools]
135+
include-package-data = false
136+
134137
[tool.setuptools.packages.find]
135138
where = ["src"]
136-
include = ["pymatgen"]
139+
include = ["pymatgen", "pymatgen.*"]
137140

138141
[tool.setuptools.package-data]
139142
"pymatgen.analysis" = ["*.yaml", "*.json", "*.csv"]

tests/core/test_structure.py

-6
Original file line numberDiff line numberDiff line change
@@ -1768,7 +1768,6 @@ def test_relax_ase_opt_kwargs(self):
17681768
assert traj[0] != traj[-1]
17691769
assert os.path.isfile(traj_file)
17701770

1771-
@pytest.mark.skip("TODO remove skip once https://github.com/materialsvirtuallab/matgl/issues/238 is resolved")
17721771
def test_calculate_m3gnet(self):
17731772
pytest.importorskip("matgl")
17741773
calculator = self.get_structure("Si").calculate()
@@ -1780,7 +1779,6 @@ def test_calculate_m3gnet(self):
17801779
assert np.linalg.norm(calculator.results["forces"]) == approx(7.8123485e-06, abs=0.2)
17811780
assert np.linalg.norm(calculator.results["stress"]) == approx(1.7861567, abs=2)
17821781

1783-
@pytest.mark.skip("TODO remove skip once https://github.com/materialsvirtuallab/matgl/issues/238 is resolved")
17841782
def test_relax_m3gnet(self):
17851783
matgl = pytest.importorskip("matgl")
17861784
struct = self.get_structure("Si")
@@ -1791,7 +1789,6 @@ def test_relax_m3gnet(self):
17911789
actual = relaxed.dynamics[key]
17921790
assert actual == val, f"expected {key} to be {val}, {actual=}"
17931791

1794-
@pytest.mark.skip("TODO remove skip once https://github.com/materialsvirtuallab/matgl/issues/238 is resolved")
17951792
def test_relax_m3gnet_fixed_lattice(self):
17961793
matgl = pytest.importorskip("matgl")
17971794
struct = self.get_structure("Si")
@@ -1800,7 +1797,6 @@ def test_relax_m3gnet_fixed_lattice(self):
18001797
assert isinstance(relaxed.calc, matgl.ext.ase.M3GNetCalculator)
18011798
assert relaxed.dynamics["optimizer"] == "BFGS"
18021799

1803-
@pytest.mark.skip("TODO remove skip once https://github.com/materialsvirtuallab/matgl/issues/238 is resolved")
18041800
def test_relax_m3gnet_with_traj(self):
18051801
pytest.importorskip("matgl")
18061802
struct = self.get_structure("Si")
@@ -2406,8 +2402,6 @@ def test_relax_ase_mol_return_traj(self):
24062402
assert traj[0] != traj[-1]
24072403
assert os.path.isfile(traj_file)
24082404

2409-
# TODO remove skip once https://github.com/tblite/tblite/issues/110 is fixed
2410-
@pytest.mark.skip("Pytorch and TBLite clash. https://github.com/materialsproject/pymatgen/pull/3060")
24112405
def test_calculate_gfnxtb(self):
24122406
pytest.importorskip("tblite")
24132407
mol_copy = self.mol.copy()

0 commit comments

Comments
 (0)