Skip to content

Commit 8ddb18c

Browse files
authored
Merge pull request #968 from payal97/main
fix typo in matminer.featurizers.structure.SiteStatsFingerprint
2 parents ae549dc + 437bb7a commit 8ddb18c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

  • src/matminer/featurizers/structure

src/matminer/featurizers/structure/sites.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ def featurize(self, s):
9393
vals = [[] for t in self._site_labels]
9494
for i, site in enumerate(s.sites):
9595
if (self.min_oxi is None or site.specie.oxi_state >= self.min_oxi) and (
96-
self.max_oxi is None or site.specie.oxi_state >= self.max_oxi
96+
self.max_oxi is None or site.specie.oxi_state <= self.max_oxi
9797
):
9898
opvalstmp = self.site_featurizer.featurize(s, i)
9999
for j, opval in enumerate(opvalstmp):

0 commit comments

Comments
 (0)