actually remove matminer from mp-api deps

Author: esoteric-ephemera

pyproject.toml

@@ -40,7 +40,6 @@ all = [
   "emmet-core[all]>=0.86.4rc1,<0.86.5",
   "fastmcp",
   "flask",
-  "matminer>=0.10.0",
   "mpcontribs-client>=5.10",
 ]
 test = [

tests/client/materials/test_similarity.py

@@ -12,12 +12,6 @@
 from mp_api.client.routes.materials.similarity import SimilarityRester
 
 
-try:
-    import matminer
-except ImportError:
-    matminer = None
-
-
 @pytest.fixture(scope="module")
 def test_struct():
     poscar = """Al2
@@ -59,21 +53,18 @@ def test_similarity_search():
 def test_similarity_vector_search(test_struct):
     rester = SimilarityRester()
 
-    # skip these tests if `matminer` is not installed
-    if matminer is not None:
-        fv = rester.fingerprint_structure(test_struct)
-        assert isinstance(fv, np.ndarray)
-        assert len(fv) == 122
-        assert isinstance(rester._fingerprinter, SimilarityScorer)
-
-        assert all(
-            isinstance(entry, SimilarityEntry)
-            and isinstance(entry.dissimilarity, float)
-            for entry in rester.find_similar(
-                test_struct,
-                top=2,
-            )
+    fv = rester.fingerprint_structure(test_struct)
+    assert isinstance(fv, np.ndarray)
+    assert len(fv) == 122
+    assert isinstance(rester._fingerprinter, SimilarityScorer)
+
+    assert all(
+        isinstance(entry, SimilarityEntry) and isinstance(entry.dissimilarity, float)
+        for entry in rester.find_similar(
+            test_struct,
+            top=2,
         )
+    )
 
     get_top = 5
     sim_entries = rester.find_similar("mp-149", top=get_top)