Env/CI: Update T019 comments [skip ci]

volkamerlab · Dec 31, 2022 · bbd7f9b · bbd7f9b
1 parent 43db85f
commit bbd7f9b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -76,10 +76,10 @@ jobs:
         run: |
           PYTEST_ARGS="--nbval-lax --current-env --dist loadscope --numprocesses 2"
 
-          # Ignored notebooks
+          # Ignore T019 under Windows, see https://github.com/volkamerlab/teachopencadd/issues/313
           PYTEST_IGNORE_T019="--ignore=teachopencadd/talktorials/T019_md_simulation/talktorial.ipynb"
 
-          # Temporarily ignored notebooks
+          # Temporarily ignored notebooks, see https://github.com/volkamerlab/teachopencadd/issues/303
           PYTEST_IGNORE_T008="--ignore=teachopencadd/talktorials/T008_query_pdb/talktorial.ipynb"
 
           if [ "$RUNNER_OS" != "Windows" ]; then

diff --git a/devtools/test_env.yml b/devtools/test_env.yml
@@ -38,8 +38,7 @@ dependencies:
   - mdtraj
   - plip
   - openmm
-  # depends on openff-toolkit -> ambertools -> not available on Windows yet!
-  # we install openmmforcefields in T019 under Linux and MacOS
+  # Dependency not included, see https://github.com/volkamerlab/teachopencadd/issues/313
   # - openmmforcefields
   - pdbfixer
   - tqdm