From c135cf3a19023ccfb1847ca92621b9ba68069663 Mon Sep 17 00:00:00 2001 From: esoteric-ephemera Date: Thu, 22 Aug 2024 16:11:02 -0700 Subject: [PATCH 01/11] update spglib symmetry dataset access and pin min version --- matminer/featurizers/structure/bonding.py | 4 ++-- pyproject.toml | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/matminer/featurizers/structure/bonding.py b/matminer/featurizers/structure/bonding.py index f22a3cdbc..148fda9cc 100644 --- a/matminer/featurizers/structure/bonding.py +++ b/matminer/featurizers/structure/bonding.py @@ -805,8 +805,8 @@ def get_equiv_sites(self, s, site): sga = SpacegroupAnalyzer(s, symprec=0.01) sg = sga.get_space_group_operations sym_data = sga.get_symmetry_dataset() - equiv_atoms = sym_data["equivalent_atoms"] - wyckoffs = sym_data["wyckoffs"] + equiv_atoms = sym_data.equivalent_atoms + wyckoffs = sym_data.wyckoffs sym_struct = SymmetrizedStructure(s, sg, equiv_atoms, wyckoffs) equivs = sym_struct.find_equivalent_sites(site) return equivs diff --git a/pyproject.toml b/pyproject.toml index 71ec89d26..50f8d71bd 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -2,7 +2,8 @@ requires = [ # pin NumPy version used in the build "numpy>=1.20.1", - "setuptools>=43.0.0" + "setuptools>=43.0.0", + "spglib>=2.5.0", ] build-backend = "setuptools.build_meta" From c9bbaa991c8092f2f126e9936b29884b0a5e323e Mon Sep 17 00:00:00 2001 From: esoteric-ephemera Date: Fri, 23 Aug 2024 09:49:52 -0700 Subject: [PATCH 02/11] move spglib version pin --- pyproject.toml | 1 - setup.py | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/pyproject.toml b/pyproject.toml index 50f8d71bd..ce428dd9b 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -3,7 +3,6 @@ requires = [ # pin NumPy version used in the build "numpy>=1.20.1", "setuptools>=43.0.0", - "spglib>=2.5.0", ] build-backend = "setuptools.build_meta" diff --git a/setup.py b/setup.py index c0e79f2de..9f9ac8faf 100644 --- a/setup.py +++ b/setup.py @@ -63,6 +63,7 @@ def local_version(version): "sympy ~= 1.11", "monty >= 2023", "pymatgen >= 2023", + "spglib>=2.5.0", ], extras_require=extras_require, classifiers=[ From 9c6173678a039ae7b58d4356219119327b3e0a7a Mon Sep 17 00:00:00 2001 From: esoteric-ephemera Date: Fri, 23 Aug 2024 11:29:06 -0700 Subject: [PATCH 03/11] ensure correct spglib installed in requirements --- requirements/ubuntu-latest_py3.10.txt | 2 +- requirements/ubuntu-latest_py3.11.txt | 2 +- requirements/ubuntu-latest_py3.9.txt | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/requirements/ubuntu-latest_py3.10.txt b/requirements/ubuntu-latest_py3.10.txt index ab2cbb6cc..d01678499 100644 --- a/requirements/ubuntu-latest_py3.10.txt +++ b/requirements/ubuntu-latest_py3.10.txt @@ -266,7 +266,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.4.0 +spglib==2.5.2 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) diff --git a/requirements/ubuntu-latest_py3.11.txt b/requirements/ubuntu-latest_py3.11.txt index 7b8853fc1..209088fa1 100644 --- a/requirements/ubuntu-latest_py3.11.txt +++ b/requirements/ubuntu-latest_py3.11.txt @@ -264,7 +264,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.4.0 +spglib==2.5.2 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) diff --git a/requirements/ubuntu-latest_py3.9.txt b/requirements/ubuntu-latest_py3.9.txt index be66219ba..6f599a535 100644 --- a/requirements/ubuntu-latest_py3.9.txt +++ b/requirements/ubuntu-latest_py3.9.txt @@ -272,7 +272,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.4.0 +spglib==2.5.2 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) From 07f9ae3abfb49363984b72d2736828b2dc7c5ce5 Mon Sep 17 00:00:00 2001 From: esoteric-ephemera Date: Fri, 23 Aug 2024 11:29:57 -0700 Subject: [PATCH 04/11] ensure correct spglib installed in requirements --- requirements/ubuntu-latest_py3.10.txt | 2 +- requirements/ubuntu-latest_py3.11.txt | 2 +- requirements/ubuntu-latest_py3.9.txt | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/requirements/ubuntu-latest_py3.10.txt b/requirements/ubuntu-latest_py3.10.txt index d01678499..86761133c 100644 --- a/requirements/ubuntu-latest_py3.10.txt +++ b/requirements/ubuntu-latest_py3.10.txt @@ -266,7 +266,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.5.2 +spglib==2.5.0 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) diff --git a/requirements/ubuntu-latest_py3.11.txt b/requirements/ubuntu-latest_py3.11.txt index 209088fa1..5da474efe 100644 --- a/requirements/ubuntu-latest_py3.11.txt +++ b/requirements/ubuntu-latest_py3.11.txt @@ -264,7 +264,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.5.2 +spglib==2.5.0 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) diff --git a/requirements/ubuntu-latest_py3.9.txt b/requirements/ubuntu-latest_py3.9.txt index 6f599a535..e5a6d6891 100644 --- a/requirements/ubuntu-latest_py3.9.txt +++ b/requirements/ubuntu-latest_py3.9.txt @@ -272,7 +272,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.5.2 +spglib==2.5.0 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) From ace1cabbbc58be295f44806455fd48bf25e8ba1f Mon Sep 17 00:00:00 2001 From: esoteric-ephemera Date: Fri, 23 Aug 2024 13:16:15 -0700 Subject: [PATCH 05/11] add a few attempts to connect to dataset downloads in case internet has bad day --- .DS_Store | Bin 0 -> 6148 bytes docs/.DS_Store | Bin 0 -> 6148 bytes matminer/.DS_Store | Bin 0 -> 6148 bytes matminer/datasets/.DS_Store | Bin 0 -> 6148 bytes matminer/datasets/tests/test_datasets.py | 11 ++++++++++- matminer/featurizers/.DS_Store | Bin 0 -> 6148 bytes matminer/utils/.DS_Store | Bin 0 -> 6148 bytes 7 files changed, 10 insertions(+), 1 deletion(-) create mode 100644 .DS_Store create mode 100644 docs/.DS_Store create mode 100644 matminer/.DS_Store create mode 100644 matminer/datasets/.DS_Store create mode 100644 matminer/featurizers/.DS_Store create mode 100644 matminer/utils/.DS_Store diff --git a/.DS_Store b/.DS_Store new file mode 100644 index 0000000000000000000000000000000000000000..ce25b109d08cdbcc8aca954372f2c4ab4abd1ab5 GIT binary patch literal 6148 zcmeHK!EVz)5S>j8-B2O<0EpvC9EpUZs^rj%8^RA@1QJjMyN+5@+Z)9WIYg0sEI+^p z&}$_)@d>C$PF(mF5Hq_QHI_@o303WmH1ozgGrQgwc{U3GtUryr04@MOewdUNoxz0J? z;Jd+roCRfAPK$mxc}-8xrA(uxA4V^d(QN46c_Q;NO!Co06(sQpQ{KEx@>tILa+=45 zstt62x7+QZdv8AXI-<9==!kjO>uq+#<8E)UX!E-d9&JAx9;R6?kLb0r*H@M>^A1gX;iHPjk*FLIykHZeU}_WVbC{NX~YN$)1`>I zl$k3A)8#lWOrCGB(x}UUnac+=D>HLLVQO`pU#N6ozD8>;1D1h018e%xrSt#k-|zqR zBKu?+unb%&2ADGl27TO;Ia?c>qqA0^zC|UWyh@`gL1T_%<o1usHA!5+m+EK%^N{eWs)sX{|a)86xC9zA>T=E5Z8SB;^XwIC=tvvr{2CLwrEEIs z_D5-jZ3BN%0e*H{G@yG*=!Ocm7p8?4zK zzb(4k;Sp~dAfJpo-a{&4-IRPBEvQ`6{Q%T1mRj>MLF;uhu)~#X4oQfKChO zHInySv$~Als*GYzukA)rr@}hLj5f1M`CL75K4zwXDPRixQ~}g%wqRG#T2sIjFa=f$ z@b@8vGe(NJp#OBB@kan)3&Y;f?{EhgDn^RAAZB1rLV*(M^oZdk9OI$#BE?)#!pZ63 z!|BdWPbkjsj`c%NCl?7?YYLbGWd)jY+2Q(sw)y;D4zep#z!dmb3bk9)W!t*w=t v<60ZSuijuUZDRU@1p)Gv?Bm$S@$pG**F+>ry|6%(C!!n@V`c%_nBaC!BQd9ImViRO zqe>@Kr5c@5Fv7MK)&c9lZ*zdxu7lMYP>Zgxc9ZoRMCXNawb^JX$#c9rOkg&ezF&!& z6)iJF7i6?0yUALTF~<>ep>tE(^jap zPaUJtOlI)aKE^ym%=5?%=h2a~=os2F{r7R4wBGfX*Q5JA>GjCF(i*4n3|(3Iu^l>t z9(K4VoxR3Wn@n|Uo2yVaftl7=f7ls#I+2~YIh$?h-H>{-(AmW=Ao`^7GTes7RBKw7 zb5|VbG8l=L@eE`9n?nrnX9H(8N6t}bbL)V0z&bE=fcFO%iP6&-E0kLYGI<36W>75y z*2Z4v36j_Dh(OK3hj7$7ijJbd1EoyjY>^f=nOB gvcO023X%-8xm*Bx8e@g1LD)Y6(gvGZ2Y%Fn57p4fwg3PC literal 0 HcmV?d00001 diff --git a/matminer/datasets/.DS_Store b/matminer/datasets/.DS_Store new file mode 100644 index 0000000000000000000000000000000000000000..86ed4f57f02cd2aa164e0ae68a8259e12834a0e8 GIT binary patch literal 6148 zcmeHKu};H447E#xtL^LF#2rAe+z_3ANU35))W|2h>d*pOZInC*U zvNdlv{6+@k?gqH6j4Hap-L38~TU?GNd0AAN_QT_L&!SAz>7ty%Bl@^~KD|GBJ^s#n zwu5&yZ)fp>ZE2sb@C&Ok)}0*ruI{e3^D5q-zVd3t7v(CK%^sce`AyDuaR!_LXW*|h zfSN554HSKJ2AlzBV99`dA7ZFrV%R8#PX~t30s#9k2fN9xox>^M6!>cj@Y>B{wU((vr&zm_^;>QoFS`DX=uXpQam!?~;2BXHj3esNG4;_RI!uRWFi|+!{B~P<#SPZ?#4Rg%mRM;AE8s}Op62`5%>2t2Tgt~p|(g1q((Raya zkbb{TjxxEty=;%TB+O^BnjdNLTjc$Q(qPq)9D9~BtHY8#YVx^z+8OaN@=YGe5mx;* zU)3MTXQxGWeY)S1^~a-T{hO8DNqr+?@{A-$SoMXh&m4Z!uxGPnvI<@A6mSYS1;z^S z`@u(J3^c|H<*NgoJOThcxTWD*zT1I5JAi@4SRp(xp;Cb=Rp=vzQ0egdIxf%{D^%$u zWM#~wD+_%>5wbe`z6>W3D0ID3z$suWpuSEi$NT@m?DM~!Ot7+$B`u2|6*Q&=iW)Dv@aU*)sDq@DfOD>^uk)i0^!- zHdY%D3qt5lI)9FRXMg!z#XTZ&{YBO#>JU*2&e+&Rv%z?tea#v=vJZ5AjS-#Fh$b|q zbS>Hj{-OeWc5R$iN(J5F>{jQOPA|HHQ9dg?ZC4EHBWHP%oK5pHM6{kC)8wP~bo5np z_JinRQf2YLWcDZn&pAA$bGoD{es5O++xhiPU0jvx{oBs#yKvpO<}6<^{Vq1(c{^mD z+u#UhXhU{1J_%>5oDLVgO!ceeSD9>%2==EJ-rvj=Fa=D3It6fNv$fiS)|vvQfGMz4 zfWHqBoH14`1^uT3jV}R!O$>WOpKA%ui4h)|vvQK&Zf$Ja+j0Kd!I;!z6n$1x$f|rGRTDgQSmF(z|u- w<@m0R;6peY$CZN16mIfdBvi literal 0 HcmV?d00001 From fdd527f8bac94df3086fb38af85d2d79059fc4dd Mon Sep 17 00:00:00 2001 From: esoteric-ephemera Date: Fri, 23 Aug 2024 13:17:14 -0700 Subject: [PATCH 06/11] remove mac hidden ds_stores and add to gitignore --- .DS_Store | Bin 6148 -> 0 bytes .gitignore | 2 ++ docs/.DS_Store | Bin 6148 -> 0 bytes matminer/.DS_Store | Bin 6148 -> 0 bytes matminer/datasets/.DS_Store | Bin 6148 -> 0 bytes matminer/featurizers/.DS_Store | Bin 6148 -> 0 bytes matminer/utils/.DS_Store | Bin 6148 -> 0 bytes 7 files changed, 2 insertions(+) delete mode 100644 .DS_Store delete mode 100644 docs/.DS_Store delete mode 100644 matminer/.DS_Store delete mode 100644 matminer/datasets/.DS_Store delete mode 100644 matminer/featurizers/.DS_Store delete mode 100644 matminer/utils/.DS_Store diff --git a/.DS_Store b/.DS_Store deleted file mode 100644 index ce25b109d08cdbcc8aca954372f2c4ab4abd1ab5..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 6148 zcmeHK!EVz)5S>j8-B2O<0EpvC9EpUZs^rj%8^RA@1QJjMyN+5@+Z)9WIYg0sEI+^p z&}$_)@d>C$PF(mF5Hq_QHI_@o303WmH1ozgGrQgwc{U3GtUryr04@MOewdUNoxz0J? z;Jd+roCRfAPK$mxc}-8xrA(uxA4V^d(QN46c_Q;NO!Co06(sQpQ{KEx@>tILa+=45 zstt62x7+QZdv8AXI-<9==!kjO>uq+#<8E)UX!E-d9&JAx9;R6?kLb0r*H@M>^A1gX;iHPjk*FLIykHZeU}_WVbC{NX~YN$)1`>I zl$k3A)8#lWOrCGB(x}UUnac+=D>HLLVQO`pU#N6ozD8>;1D1h018e%xrSt#k-|zqR zBKu?+unb%&2ADGl27TO;Ia?c>qqA0^zC|UWyh@`gL1T_%<o1usHA!5+m+EK%^N{eWs)sX{|a)86xC9zA>T=E5Z8SB;^XwIC=tvvr{2CLwrEEIs z_D5-jZ3BN%0e*H{G@yG*=!Ocm7p8?4zK zzb(4k;Sp~dAfJpo-a{&4-IRPBEvQ`6{Q%T1mRj>MLF;uhu)~#X4oQfKChO zHInySv$~Als*GYzukA)rr@}hLj5f1M`CL75K4zwXDPRixQ~}g%wqRG#T2sIjFa=f$ z@b@8vGe(NJp#OBB@kan)3&Y;f?{EhgDn^RAAZB1rLV*(M^oZdk9OI$#BE?)#!pZ63 z!|BdWPbkjsj`c%NCl?7?YYLbGWd)jY+2Q(sw)y;D4zep#z!dmb3bk9)W!t*w=t v<60ZSuijuUZDRU@1p)Gv?Bm$S@$pG**F+>ry|6%(C!!n@V`c%_nBaC!BQd9ImViRO zqe>@Kr5c@5Fv7MK)&c9lZ*zdxu7lMYP>Zgxc9ZoRMCXNawb^JX$#c9rOkg&ezF&!& z6)iJF7i6?0yUALTF~<>ep>tE(^jap zPaUJtOlI)aKE^ym%=5?%=h2a~=os2F{r7R4wBGfX*Q5JA>GjCF(i*4n3|(3Iu^l>t z9(K4VoxR3Wn@n|Uo2yVaftl7=f7ls#I+2~YIh$?h-H>{-(AmW=Ao`^7GTes7RBKw7 zb5|VbG8l=L@eE`9n?nrnX9H(8N6t}bbL)V0z&bE=fcFO%iP6&-E0kLYGI<36W>75y z*2Z4v36j_Dh(OK3hj7$7ijJbd1EoyjY>^f=nOB gvcO023X%-8xm*Bx8e@g1LD)Y6(gvGZ2Y%Fn57p4fwg3PC diff --git a/matminer/datasets/.DS_Store b/matminer/datasets/.DS_Store deleted file mode 100644 index 86ed4f57f02cd2aa164e0ae68a8259e12834a0e8..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 6148 zcmeHKu};H447E#xtL^LF#2rAe+z_3ANU35))W|2h>d*pOZInC*U zvNdlv{6+@k?gqH6j4Hap-L38~TU?GNd0AAN_QT_L&!SAz>7ty%Bl@^~KD|GBJ^s#n zwu5&yZ)fp>ZE2sb@C&Ok)}0*ruI{e3^D5q-zVd3t7v(CK%^sce`AyDuaR!_LXW*|h zfSN554HSKJ2AlzBV99`dA7ZFrV%R8#PX~t30s#9k2fN9xox>^M6!>cj@Y>B{wU((vr&zm_^;>QoFS`DX=uXpQam!?~;2BXHj3esNG4;_RI!uRWFi|+!{B~P<#SPZ?#4Rg%mRM;AE8s}Op62`5%>2t2Tgt~p|(g1q((Raya zkbb{TjxxEty=;%TB+O^BnjdNLTjc$Q(qPq)9D9~BtHY8#YVx^z+8OaN@=YGe5mx;* zU)3MTXQxGWeY)S1^~a-T{hO8DNqr+?@{A-$SoMXh&m4Z!uxGPnvI<@A6mSYS1;z^S z`@u(J3^c|H<*NgoJOThcxTWD*zT1I5JAi@4SRp(xp;Cb=Rp=vzQ0egdIxf%{D^%$u zWM#~wD+_%>5wbe`z6>W3D0ID3z$suWpuSEi$NT@m?DM~!Ot7+$B`u2|6*Q&=iW)Dv@aU*)sDq@DfOD>^uk)i0^!- zHdY%D3qt5lI)9FRXMg!z#XTZ&{YBO#>JU*2&e+&Rv%z?tea#v=vJZ5AjS-#Fh$b|q zbS>Hj{-OeWc5R$iN(J5F>{jQOPA|HHQ9dg?ZC4EHBWHP%oK5pHM6{kC)8wP~bo5np z_JinRQf2YLWcDZn&pAA$bGoD{es5O++xhiPU0jvx{oBs#yKvpO<}6<^{Vq1(c{^mD z+u#UhXhU{1J_%>5oDLVgO!ceeSD9>%2==EJ-rvj=Fa=D3It6fNv$fiS)|vvQfGMz4 zfWHqBoH14`1^uT3jV}R!O$>WOpKA%ui4h)|vvQK&Zf$Ja+j0Kd!I;!z6n$1x$f|rGRTDgQSmF(z|u- w<@m0R;6peY$CZN16mIfdBvi From e442701af5a92a389f4962ac3179b7c4ed2935d5 Mon Sep 17 00:00:00 2001 From: esoteric-ephemera Date: Mon, 26 Aug 2024 09:53:17 -0700 Subject: [PATCH 07/11] precommit --- matminer/datasets/tests/test_datasets.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/matminer/datasets/tests/test_datasets.py b/matminer/datasets/tests/test_datasets.py index 3ba701f56..ea3e108fb 100644 --- a/matminer/datasets/tests/test_datasets.py +++ b/matminer/datasets/tests/test_datasets.py @@ -82,9 +82,9 @@ def universal_dataset_check( download_page = requests.head(self.dataset_dict[dataset_name]["url"]) if download_page.ok: break - if iconnect < max_connect_attempts-1: + if iconnect < max_connect_attempts - 1: sleep(5) - + self.assertTrue(download_page.ok) From f12be3a181a868af6a3d24f6e1330839176d4788 Mon Sep 17 00:00:00 2001 From: esoteric-ephemera Date: Mon, 23 Sep 2024 09:06:05 -0700 Subject: [PATCH 08/11] ensure backwards compat with older spglib, loosen lower pin on spglib version, upper pin on numpy --- matminer/featurizers/structure/bonding.py | 4 ++-- pyproject.toml | 2 +- requirements/ubuntu-latest_py3.10.txt | 2 +- requirements/ubuntu-latest_py3.11.txt | 2 +- requirements/ubuntu-latest_py3.9.txt | 2 +- setup.py | 1 - 6 files changed, 6 insertions(+), 7 deletions(-) diff --git a/matminer/featurizers/structure/bonding.py b/matminer/featurizers/structure/bonding.py index 148fda9cc..36183727f 100644 --- a/matminer/featurizers/structure/bonding.py +++ b/matminer/featurizers/structure/bonding.py @@ -805,8 +805,8 @@ def get_equiv_sites(self, s, site): sga = SpacegroupAnalyzer(s, symprec=0.01) sg = sga.get_space_group_operations sym_data = sga.get_symmetry_dataset() - equiv_atoms = sym_data.equivalent_atoms - wyckoffs = sym_data.wyckoffs + equiv_atoms = getattr(sym_data,"equivalent_atoms",sym_data["equivalent_atoms"]) + wyckoffs = getattr(sym_data,"wyckoffs",sym_data["wyckoffs"]) sym_struct = SymmetrizedStructure(s, sg, equiv_atoms, wyckoffs) equivs = sym_struct.find_equivalent_sites(site) return equivs diff --git a/pyproject.toml b/pyproject.toml index ce428dd9b..77d85ff41 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -1,7 +1,7 @@ [build-system] requires = [ # pin NumPy version used in the build - "numpy>=1.20.1", + "numpy>=1.20.1, <2", "setuptools>=43.0.0", ] build-backend = "setuptools.build_meta" diff --git a/requirements/ubuntu-latest_py3.10.txt b/requirements/ubuntu-latest_py3.10.txt index 86761133c..ab2cbb6cc 100644 --- a/requirements/ubuntu-latest_py3.10.txt +++ b/requirements/ubuntu-latest_py3.10.txt @@ -266,7 +266,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.5.0 +spglib==2.4.0 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) diff --git a/requirements/ubuntu-latest_py3.11.txt b/requirements/ubuntu-latest_py3.11.txt index 5da474efe..7b8853fc1 100644 --- a/requirements/ubuntu-latest_py3.11.txt +++ b/requirements/ubuntu-latest_py3.11.txt @@ -264,7 +264,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.5.0 +spglib==2.4.0 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) diff --git a/requirements/ubuntu-latest_py3.9.txt b/requirements/ubuntu-latest_py3.9.txt index e5a6d6891..be66219ba 100644 --- a/requirements/ubuntu-latest_py3.9.txt +++ b/requirements/ubuntu-latest_py3.9.txt @@ -272,7 +272,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.5.0 +spglib==2.4.0 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) diff --git a/setup.py b/setup.py index 9f9ac8faf..c0e79f2de 100644 --- a/setup.py +++ b/setup.py @@ -63,7 +63,6 @@ def local_version(version): "sympy ~= 1.11", "monty >= 2023", "pymatgen >= 2023", - "spglib>=2.5.0", ], extras_require=extras_require, classifiers=[ From 45a005a6710e9158ef322853c485ab94aa1b84a7 Mon Sep 17 00:00:00 2001 From: esoteric-ephemera Date: Mon, 23 Sep 2024 09:06:24 -0700 Subject: [PATCH 09/11] precommit --- matminer/featurizers/structure/bonding.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/matminer/featurizers/structure/bonding.py b/matminer/featurizers/structure/bonding.py index 36183727f..69b1ecc9e 100644 --- a/matminer/featurizers/structure/bonding.py +++ b/matminer/featurizers/structure/bonding.py @@ -805,8 +805,8 @@ def get_equiv_sites(self, s, site): sga = SpacegroupAnalyzer(s, symprec=0.01) sg = sga.get_space_group_operations sym_data = sga.get_symmetry_dataset() - equiv_atoms = getattr(sym_data,"equivalent_atoms",sym_data["equivalent_atoms"]) - wyckoffs = getattr(sym_data,"wyckoffs",sym_data["wyckoffs"]) + equiv_atoms = getattr(sym_data, "equivalent_atoms", sym_data["equivalent_atoms"]) + wyckoffs = getattr(sym_data, "wyckoffs", sym_data["wyckoffs"]) sym_struct = SymmetrizedStructure(s, sg, equiv_atoms, wyckoffs) equivs = sym_struct.find_equivalent_sites(site) return equivs From eed6f93b1bcefcdfb8fdb1beabb43648fd62939d Mon Sep 17 00:00:00 2001 From: esoteric-ephemera Date: Wed, 25 Sep 2024 10:34:35 -0700 Subject: [PATCH 10/11] repin spglib == 2.5, add read permissions for workflow actions (somehow GH token not being read --- .github/workflows/test.yml | 3 +++ requirements/ubuntu-latest_py3.10.txt | 2 +- requirements/ubuntu-latest_py3.11.txt | 2 +- requirements/ubuntu-latest_py3.9.txt | 2 +- 4 files changed, 6 insertions(+), 3 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index e362429ea..2c5d40bd3 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -3,6 +3,9 @@ # The ext package is also only tested in this workflow. Coverage is also computed based on this platform. name: Testing +permissions: + contents: read + env: PYTEST_ADDOPTS: "--color=yes" diff --git a/requirements/ubuntu-latest_py3.10.txt b/requirements/ubuntu-latest_py3.10.txt index ab2cbb6cc..86761133c 100644 --- a/requirements/ubuntu-latest_py3.10.txt +++ b/requirements/ubuntu-latest_py3.10.txt @@ -266,7 +266,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.4.0 +spglib==2.5.0 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) diff --git a/requirements/ubuntu-latest_py3.11.txt b/requirements/ubuntu-latest_py3.11.txt index 7b8853fc1..5da474efe 100644 --- a/requirements/ubuntu-latest_py3.11.txt +++ b/requirements/ubuntu-latest_py3.11.txt @@ -264,7 +264,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.4.0 +spglib==2.5.0 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) diff --git a/requirements/ubuntu-latest_py3.9.txt b/requirements/ubuntu-latest_py3.9.txt index be66219ba..e5a6d6891 100644 --- a/requirements/ubuntu-latest_py3.9.txt +++ b/requirements/ubuntu-latest_py3.9.txt @@ -272,7 +272,7 @@ soupsieve==2.5 # via beautifulsoup4 sparse==0.15.4 # via dscribe -spglib==2.4.0 +spglib==2.5.0 # via pymatgen sphinx==7.3.7 # via matminer (setup.py) From 8f372833fb2427a790ef7af8349f511e8a3d1faf Mon Sep 17 00:00:00 2001 From: Matthew Evans <7916000+ml-evs@users.noreply.github.com> Date: Mon, 30 Sep 2024 14:08:03 +0200 Subject: [PATCH 11/11] Update .github/workflows/test.yml --- .github/workflows/test.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 2c5d40bd3..7f31e5bc6 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -5,7 +5,7 @@ name: Testing permissions: contents: read - + env: PYTEST_ADDOPTS: "--color=yes"