Skip to content

Commit a7452ee

Browse files
authored
Doc: fix typos in notebooks (#1977)
* fix typos * fix typos in example_errors
1 parent de44941 commit a7452ee

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

documentation/GettingStarted.ipynb

+3-3
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
"metadata": {},
1515
"source": [
1616
"## Model Compilation\n",
17-
"Before simulations can be run, the model must be imported and compiled. In this process, AMICI performs all symbolic manipulations that later enable scalable simulations and efficient sensitivity computation. The first towards model compilation is the creation of an [SbmlImporter](https://amici.readthedocs.io/en/latest/generated/amici.sbml_import.SbmlImporter.html) instance, which requires an SBML Document that specifies the model using the [Systems Biology Markup Language (SBML)](http://sbml.org/Main_Page). \n",
17+
"Before simulations can be run, the model must be imported and compiled. In this process, AMICI performs all symbolic manipulations that later enable scalable simulations and efficient sensitivity computation. The first step towards model compilation is the creation of an [SbmlImporter](https://amici.readthedocs.io/en/latest/generated/amici.sbml_import.SbmlImporter.html) instance, which requires an SBML Document that specifies the model using the [Systems Biology Markup Language (SBML)](http://sbml.org/Main_Page). \n",
1818
"\n",
1919
"For the purpose of this tutorial, we will use `model_steadystate_scaled.xml`, which is contained in the same directory as this notebook."
2020
]
@@ -52,7 +52,7 @@
5252
"metadata": {},
5353
"source": [
5454
"## Loading the model module\n",
55-
"To run simulations, we need to instantiate [amici.Model](https://amici.readthedocs.io/en/latest/generated/amici.amici.Model.html) and [amici.Solver](https://amici.readthedocs.io/en/latest/generated/amici.amici.Solver.html) instances. As simulations requires instances matching the imported model, they have to be imported from the generated model module. "
55+
"To run simulations, we need to instantiate [amici.Model](https://amici.readthedocs.io/en/latest/generated/amici.amici.Model.html) and [amici.Solver](https://amici.readthedocs.io/en/latest/generated/amici.amici.Solver.html) instances. As simulations require instances matching the imported model, they have to be imported from the generated model module. "
5656
]
5757
},
5858
{
@@ -205,7 +205,7 @@
205205
"name": "python",
206206
"nbconvert_exporter": "python",
207207
"pygments_lexer": "ipython3",
208-
"version": "3.8.5"
208+
"version": "3.8.10"
209209
},
210210
"toc": {
211211
"base_numbering": 1,

python/examples/example_errors.ipynb

+6-6
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@
201201
"source": [
202202
"**What happened?**\n",
203203
"\n",
204-
"The forward simulation failed because AMICI the solver exceeded the maximum number of steps. Unlike in the previous case of `mxstep steps taken before reaching tout` (see above), here we got several additional warnings that the current step size $h$ is numerically zero.\n",
204+
"The forward simulation failed because the AMICI solver exceeded the maximum number of steps. Unlike in the previous case of `mxstep steps taken before reaching tout` (see above), here we got several additional warnings that the current step size $h$ is numerically zero.\n",
205205
"\n",
206206
"**How to address?**\n",
207207
"\n",
@@ -502,7 +502,7 @@
502502
"[![](https://mermaid.ink/img/pako:eNpdkrFuwyAQhl8FIWVL1MwMndKFtd1wBmJIg2IDwucCivLuxQp2zvaA_P1398Od7kFbpzRl9DdIfyM_p8aS8t3FnZHW2QGkheH8Er3wjHgZZK9Bh1kFAYyA6XXldBTpyIixBozsSHGAJSQSWwlccEa4bN3FSFu1KCIjOvmgh8GUF8y1yoGYDkZU-lBQ5SwyI-4y6PAnL52esl-B3a68pDZDDofPhfyKYEVLacSVERdGXFchzYAu59iBYweOHTh2qBAxvLupI3s9eJ41pndqGdOqvYXThv2G7xuOiBf7jHMzNsr41oyvzNgv0z3tdeilUWXzHlOooXDTvW4oK79KX-XYQUMb-yypcgT3nW1LGYRR7-noVVmhk5FlZ3vKrrIbFvVLGXChis9_j9jNUw?type=png)](https://mermaid.live/edit#pako:eNpdkrFuwyAQhl8FIWVL1MwMndKFtd1wBmJIg2IDwucCivLuxQp2zvaA_P1398Od7kFbpzRl9DdIfyM_p8aS8t3FnZHW2QGkheH8Er3wjHgZZK9Bh1kFAYyA6XXldBTpyIixBozsSHGAJSQSWwlccEa4bN3FSFu1KCIjOvmgh8GUF8y1yoGYDkZU-lBQ5SwyI-4y6PAnL52esl-B3a68pDZDDofPhfyKYEVLacSVERdGXFchzYAu59iBYweOHTh2qBAxvLupI3s9eJ41pndqGdOqvYXThv2G7xuOiBf7jHMzNsr41oyvzNgv0z3tdeilUWXzHlOooXDTvW4oK79KX-XYQUMb-yypcgT3nW1LGYRR7-noVVmhk5FlZ3vKrrIbFvVLGXChis9_j9jNUw)\n",
503503
"\n",
504504
"Always look for the most basic (furthest up) model quantities.\n",
505-
"In cases where there non-finite values occur in expressions further down, rerunning the simulation after calling `Model.setAlwaysCheckFinite(True)` may give some further hints on where the issue originates.\n",
505+
"In cases where non-finite values occur in expressions further down, rerunning the simulation after calling `Model.setAlwaysCheckFinite(True)` may give some further hints on where the issue originates.\n",
506506
"\n",
507507
"The `NaN` in $w$ occurred for `flux_v7_v_6` (see error log), i.e., when computing the reaction flux for reaction `v7_v_6`. As $w$ only depends on $(t, p, k, x)$ and no non-finite values have been reported for those, the issue has to be in the respective flux equation.\n",
508508
"\n",
@@ -529,7 +529,7 @@
529529
"id": "9f49a00a",
530530
"metadata": {},
531531
"source": [
532-
"What could go wrong? We can obtain `NaN` from any of these symbols symbols being `NaN`, or through division by zero.\n",
532+
"What could go wrong? We can obtain `NaN` from any of these symbols being `NaN`, or through division by zero.\n",
533533
"\n",
534534
"Let's let's check the denominator first: $$(A\\_state^2 + Kp^2)*(Kd^{n\\_par} + Z\\_state^{n\\_par})$$\n",
535535
"\n",
@@ -670,7 +670,7 @@
670670
"source": [
671671
"**What happened?**\n",
672672
"\n",
673-
"AMICI failed to compute steadystate sensitivities, because it was not able to factorize the Jacobian.\n",
673+
"AMICI failed to compute steady-state sensitivities, because it was not able to factorize the Jacobian.\n",
674674
"\n",
675675
"**How to address?**\n",
676676
"\n",
@@ -815,7 +815,7 @@
815815
"source": [
816816
"**What happened?**\n",
817817
"\n",
818-
"All given experimental conditions require pre-equilibration, i.e., finding a steady state. AMICI first tries find a steady state using the Newton solver, if that fails, it tries simulating until steady state, if that also failes, it tries the Newton solver from the end of the simulation. In this case, all three failed. Neither Newton's method nor simulation yielded a steadystate satisfying the required tolerances.\n",
818+
"All given experimental conditions require pre-equilibration, i.e., finding a steady state. AMICI first tries to find a steady state using the Newton solver, if that fails, it tries simulating until steady state, if that also failes, it tries the Newton solver from the end of the simulation. In this case, all three failed. Neither Newton's method nor simulation yielded a steady state satisfying the required tolerances.\n",
819819
"\n",
820820
"This can also be seen in `ReturnDataView.preeq_status` (the three statuses corresponds to Newton \\#1, Simulation, Newton \\#2):"
821821
]
@@ -842,7 +842,7 @@
842842
"\n",
843843
"1. Stricter integration tolerances (preferred if affordable - higher accuracy, but generally slower)\n",
844844
"\n",
845-
"2. Looser steadystate tolerances (lower accuracy, generally faster)\n",
845+
"2. Looser steady-state tolerances (lower accuracy, generally faster)\n",
846846
"\n",
847847
"3. Increase the number of allowed steps for Newton's method\n",
848848
"\n",

0 commit comments

Comments
 (0)