smmorey/curriculum updates (#156)

* reset cel output & renamed hillslope_functions notebook

* Updated diffusion and functino notebooks

* Update linters and actions (#157)

* Update pre-commit hooks

* Clean lint with updated pre-commit hooks

* Allow E704, multiple definitions per line

* Manually clean lint found by flake8

* Make test-notebooks job cancelable

* Use setup-python action instead of setup-miniconda

* Use the conda-forge channel to find nbmake

* Also use conda-forge channel for requirements

* Clean lint with pre-commit hooks

* Replace invalid API key with demo key

* Use python=3.13

* Use latest versions of landlab and pymt

* Clean lint

* Disable nbqa-isort, fighting with black

* Clean lint with pre-commit hooks

* Disable flake8 rule F821

It's giving incorrect info about undefined variable names that are
defined in cells upstream.

* Change variable name from l to lx

flake8 was warning about an ambiguous variable name, since "l"
could be interpreted as "1".

* Skip tests on hillslope notebooks

We can come back later to get them working with tests.

---------

Co-authored-by: Mark Piper <[email protected]>

Author: smmorey

.pre-commit-config.yaml

@@ -27,9 +27,9 @@ repos:
   hooks:
     - id: nbqa-pyupgrade
       args: ["--py38-plus"]
-    - id: nbqa-isort
+    # - id: nbqa-isort
     - id: nbqa-flake8
-      args: ["--extend-ignore=E402,F811"]
+      args: ["--extend-ignore=E402,F811,F821"]
       exclude: |
           (?x)^(
               lessons/landlab/landlab-terrainbento/|