Skip to content

Commit cef0451

Browse files
authored
Merge pull request #268 from OpenFreeEnergy/fix/notebook_tests
fix failing cells in example notebooks
2 parents b230cd6 + fbe6a86 commit cef0451

File tree

5 files changed

+268
-241
lines changed

5 files changed

+268
-241
lines changed

.github/workflows/openfe-doc-build.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ defaults:
2121
shell: bash -leo pipefail {0}
2222

2323
jobs:
24-
test-conda-build:
24+
test-openfe-docs-build:
2525
runs-on: ubuntu-latest
2626

2727
steps:

abfe_tutorial/abfe_tutorial.ipynb

Lines changed: 11 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -912,44 +912,22 @@
912912
},
913913
{
914914
"cell_type": "code",
915-
"execution_count": 19,
915+
"execution_count": null,
916916
"id": "f62f69a1-09c0-4a4a-9b37-9663b51a75ac",
917917
"metadata": {},
918918
"outputs": [],
919919
"source": [
920-
"import gzip\n",
921-
"import json\n",
922-
"import gufe\n",
920+
"# import gzip\n",
921+
"# import json\n",
922+
"# import gufe\n",
923923
"\n",
924-
"outfile = \"abfe_results/toluene_results.json\"\n",
925-
"with open(outfile) as stream:\n",
926-
" results = json.load(stream)\n",
927-
" estimate = results['estimate']\n",
928-
" uncertainty = results['uncertainty']"
929-
]
930-
},
931-
{
932-
"cell_type": "code",
933-
"execution_count": 20,
934-
"id": "3b459b28-a4dc-4fa9-a961-b106c45d79ce",
935-
"metadata": {},
936-
"outputs": [
937-
{
938-
"data": {
939-
"text/plain": [
940-
"{'magnitude': -4.671169105743085,\n",
941-
" 'unit': 'kilocalorie_per_mole',\n",
942-
" ':is_custom:': True,\n",
943-
" 'pint_unit_registry': 'openff_units'}"
944-
]
945-
},
946-
"execution_count": 20,
947-
"metadata": {},
948-
"output_type": "execute_result"
949-
}
950-
],
951-
"source": [
952-
"estimate"
924+
"# outfile = \"abfe_results/toluene_results.json\"\n",
925+
"# with open(outfile) as stream:\n",
926+
"# results = json.load(stream)\n",
927+
"# estimate = results['estimate']\n",
928+
"# uncertainty = results['uncertainty']\n",
929+
"\n",
930+
"# estimate"
953931
]
954932
}
955933
],

0 commit comments

Comments
 (0)