ruff enable PIE (flake8-pie)

fix 3 PIE804 Unnecessary `dict` kwargs
materialsproject · May 6, 2023 · 490fbc5 · 490fbc5
1 parent 359fed2
commit 490fbc5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/pymatgen/io/ase.py b/pymatgen/io/ase.py
@@ -87,11 +87,11 @@ def get_atoms(structure, **kwargs):
         charges = structure.site_properties["final_charge"] if "final_charge" in structure.site_properties else None
         if magmoms or charges:
             if magmoms and charges:
-                calc = SinglePointDFTCalculator(atoms, **{"magmoms": magmoms, "charges": charges})
+                calc = SinglePointDFTCalculator(atoms, magmoms=magmoms, charges=charges)
             elif magmoms:
-                calc = SinglePointDFTCalculator(atoms, **{"magmoms": magmoms})
+                calc = SinglePointDFTCalculator(atoms, magmoms=magmoms)
             elif charges:
-                calc = SinglePointDFTCalculator(atoms, **{"charges": charges})
+                calc = SinglePointDFTCalculator(atoms, charges=charges)
             atoms.calc = calc
 
         # Get the oxidation states from the structure

diff --git a/pyproject.toml b/pyproject.toml
@@ -32,6 +32,7 @@ select = [
   "E",   # pycodestyle
   "F",   # pyflakes
   "I",   # isort
+  "PIE", # flake8-pie
   "PLE", # pylint error
   "PLW", # pylint warning
   "Q",   # flake8-quotes