Skip to content

docs: refresh reference manual#269

Open
galjos wants to merge 1 commit into
devfrom
feature/docs-reference-refresh
Open

docs: refresh reference manual#269
galjos wants to merge 1 commit into
devfrom
feature/docs-reference-refresh

Conversation

@galjos

@galjos galjos commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add quick-start, examples, and troubleshooting pages
  • refresh reference/setup docs and feature list
  • document build options and contributor formatting

Validation

  • make html
  • git diff --check

@galjos galjos added the documentation Improvements or additions to documentation label Jun 12, 2026
@galjos galjos requested review from 97gamjak and ape33 June 12, 2026 12:27

@ape33 ape33 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice update of the ref manual!

Comment on lines +50 to +55
| DFTB+
| Turbomole
| PySCF
| MACE-MP and MACE-OFF
| ASE-DFTB+
| ASE-xTB

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest adding links, just like in the reference manual of the input file

Comment on lines +61 to +88
Integrator:

| Velocity Verlet

1) Velocity Verlet
Thermostats:

Thermostats
===========
| Langevin
| Berendsen
| stochastic velocity rescaling
| Nose-Hoover chain

1) Langevin Thermostat
2) Berendsen Thermostat
3) Velocity Rescaling Thermostat
4) Nose-Hoover Thermostat
Manostats:

Manostats
=========
| Berendsen
| stochastic cell rescaling

1) Berendsen Manostat
2) Stochastic Rescaling Manostat
Cell coupling modes:

Isotropicity
============
| isotropic
| semi-isotropic
| anisotropic cell lengths
| full anisotropic cell lengths and angles

All calculation schemes of any MD runner can be performed with triclinic cells
Constraints:

1) Isotropic
2) Semi-Isotropic
3) Anisotropic (only cell lengths)
4) Full Anisotropic (cell lengths and angles)
| SHAKE/RATTLE
| M-SHAKE
| distance constraints

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When looking at the rendered html file I don't like that the spacing before and after the category are the same, e.g. white space before and after Manostats. I would also like more highlighting/differentiation between the category (e.g. Manostats) and the associated items (e.g. Berendsen stochastic cell rescaling)
either with explicit bullet points or bold font or font size

Planned Items
*************

The following items are not documented as supported features in this release:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would somehow add that these items are planned, because phrasing it like this sounds just like an info that they are not featured, however they are indeed planned;
example suggestion:
The following items are planned for future releases.


| ``jobtype = mm-md`` for a molecular-mechanics MD simulation
| ``start_file = input_h2o.rst`` for the initial structure
| default ``moldescriptor_file`` and ``guff_file`` setup names

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest "setup files" instead of "setup names"

Comment on lines +198 to +202
The first line of each block is the number of atoms in the reference geometry.
The second line must contain ``moltype = <id>;`` and is parsed with the same
command syntax as the input file. It is followed by one atom line per reference
atom: atom name and Cartesian coordinates. The atom names must match the atom
names of the referenced molecule type.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add that the moltype comes from the moldescriptor.dat file and also link it

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add a note that when the force-field keyword in the input file is set to on or bonded, the topology file and the parameter file become mandatory; also add corresponding links
Moreover, the topology file becomes mandatory if the shake keyword is set to on --> add link


Some setup files are only required for specific modes:

| ``topology_file`` is required when topology-defined constraints or bonded force-field terms are used.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is required if force-field is set to bonded or on; or shake is set to on

Some setup files are only required for specific modes:

| ``topology_file`` is required when topology-defined constraints or bonded force-field terms are used.
| ``parameter_file`` is required for force-field simulations.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is required if force-field is set to bonded or on

| ``qm-rpmd``
| ``mm-opt``

These values are case-insensitive and dashes are treated like underscores.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

omit this remark here and instead add this information as a general info to the referenceManual->InputFile

Comment on lines 1130 to 1132
2. **on** - SHAKE for bond constraints defined in the :ref:`topologyFile` will be applied.

3. **shake** - SHAKE for bond constraints defined in the :ref:`topologyFile` will be applied.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please fuse these two options into one bullet point

@ape33 ape33 linked an issue Jun 23, 2026 that may be closed by this pull request
@ape33 ape33 removed a link to an issue Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants