refactor: resolve ruff check F821 for undefined name (#2062)

* refactor: resolve ruff check F821 for undefined name

* Update test_gwf_lak_wetlakbedarea01.py

* Update test_gwe_esl02.py

make codespell happy

* ruff

---------

Co-authored-by: Eric Morway <emorway@usgs.gov>
Co-authored-by: langevin-usgs <langevin@usgs.gov>

autotest/test_gwe_esl02.py

@@ -340,7 +340,8 @@ def check_output(idx, test):
         "Grid cell temperatures do not reflect the expected difference"
         "in stress period "
     )
-    assert np.isclose(np.sum(temps[-1]), ncol[idx]), msg0 + str(index)
+    answer = ncol[idx]
+    assert np.isclose(np.sum(temps[-1]), answer), msg0 + str(idx)
 
 
 # - No need to change any code below

autotest/test_gwe_obs.py

@@ -488,7 +488,7 @@ def check_output(idx, test):
         # return out of function
         return None
     else:
-        assert error_count == 0, errmsg0
+        assert error_count == 0
 
     name = cases[idx]
     gwename1 = "gwe-" + name + "-1"

autotest/test_gwf_lak_wetlakbedarea01.py

@@ -141,12 +141,6 @@ def calc_qSat(top, bot, thk):
         else:
             y = 1.0
 
-    else:
-        if x < bot:
-            y = 0.0
-        else:
-            y = 1.0
-
     return y
 
 

autotest/test_gwf_laket.py

@@ -286,6 +286,7 @@ def check_output(idx, test):
         stage_compare = True
 
     test.success = True
+    msg = ""
     if not evap_compare:
         test.success = False
         msg += " Lake evaporation comparison failed."

doc/mf6io/mf6ivar/mf6ivar.py

@@ -157,7 +157,7 @@ def parse_mf6var_file(fname):
                 else:
                     key = name
                 if key in vardict:
-                    raise ValueError(f"Variable already exists in dictionary: {k}")
+                    raise ValueError(f"Variable already exists in dictionary: {key}")
                 vardict[key] = vd
             vd = {}
             continue
@@ -233,9 +233,7 @@ def block_entry(varname, block, vardict, prefix="  "):
     elif " " in vtype:
         vtype = vtype.split(" ", 1)[0]
     if vtype not in VALID_TYPES:
-        raise ValueError(
-            f"{fname}: {key}: {vtype!r} is not a valid type from {VALID_TYPES}"
-        )
+        raise ValueError(f"{key}: {vtype!r} is not a valid type from {VALID_TYPES}")
 
     # record or recarray
     if v["type"].startswith("rec"):

ruff.toml

@@ -11,7 +11,6 @@ select = [
 ignore = [
     "E722", # do not use bare `except`
     "E741", # ambiguous variable name
-    "F821", # undefined name TODO FIXME
     "F841", # local variable assigned but never used
 ]