Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update kim_SW_Si.ipynb #202

Merged
merged 1 commit into from
Feb 25, 2025
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions docs/source/tutorials/kim_SW_Si.ipynb
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@
" use KLIFF to train potentials. It should not be used to train any potential for real\n",
" simulations.</p></div>\n",
"\n",
"Let's first import the modules that will be used in this example.\n",
"Let's first import the modules that will be used in this example. In python, we write:\n",
"\n"
]
},
Expand Down Expand Up @@ -75,7 +75,7 @@
"## Model\n",
"\n",
"We first create a KIM model for the SW potential, and print out all the available\n",
"parameters that can be optimized (we call this ``model parameters``).\n",
"parameters that can be optimized (we call these the ``model parameters``). Continuing in our python script we write\n",
"\n"
]
},
Expand Down Expand Up @@ -230,7 +230,7 @@
"source": [
"Here, we tell KLIFF to fit four parameters ``B``, ``gamma``, ``sigma``, and ``A`` of the\n",
"SW model. The information for each fitting parameter should be provided as a list of\n",
"list, where the size of the outer list should be equal to the ``size`` of the parameter\n",
"lists, where the size of the outer list should be equal to the ``size`` of the parameter\n",
"given by ``model.echo_model_params()``. For each inner list, you can provide either one,\n",
"two, or three items.\n",
"\n",
Expand Down Expand Up @@ -304,9 +304,9 @@
"## Calculator\n",
"\n",
":class:`~kliff.calculator.Calculator` is the central agent that exchanges information\n",
"and orchestrate the operation of the fitting process. It calls the model to compute the\n",
"and orchestrates the operation of the fitting process. It calls the model to compute the\n",
"energy and forces and provide this information to the `Loss function`_ (discussed below)\n",
"to compute the loss. It also grabs the parameters from the optimizer and update the\n",
"to compute the loss. It also grabs the parameters from the optimizer and updates the\n",
"parameters stored in the model so that the up-to-date parameters are used the next time\n",
"the model is evaluated to compute the energy and forces. The calculator can be created\n",
"by:\n",
Expand Down Expand Up @@ -542,7 +542,7 @@
"metadata": {},
"source": [
"The minimization stops after running for 27 steps. After the minimization, we'd better\n",
"save the model, which can be loaded later for the purpose to do a retraining or\n",
"save the model, which can be loaded later for the purpose of retraining or for function\n",
"evaluations. If satisfied with the fitted model, you can also write it as a KIM model\n",
"that can be used with LAMMPS_, GULP_, ASE_, etc. via the kim-api_.\n",
"\n"
Expand Down Expand Up @@ -609,7 +609,7 @@
"``kliff_model.pkl`` on the disk, and the third line writes out a KIM potential named\n",
"``SW_StillingerWeber_1985_Si__MO_405512056662_006_kliff_trained``.\n",
"\n",
".. seealso::\n",
"- SeeAlso: \n",
" For information about how to load a saved model, see `doc.modules`.\n",
"\n",
"\n",
Expand Down
Loading