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

Central chapter for thermostats and integrators #3913

Closed
jngrad opened this issue Sep 25, 2020 · 1 comment · Fixed by #4080
Closed

Central chapter for thermostats and integrators #3913

jngrad opened this issue Sep 25, 2020 · 1 comment · Fixed by #4080

Comments

@jngrad
Copy link
Member

jngrad commented Sep 25, 2020

The number of thermostats and integrators has increased in 4.2, and their documentation has improved substantially. They are currently documented in chapters "3. Setting up the system" and "5. Running the simulation", respectively. However, they are tightly coupled to each other. I think the user guide would be more intuitive if thermostats were moved into chapter "5. Running the simulation".

The restructured user guide would look like this:

  1. Setting up the system
    1. Setting global variables in Python
    2. Cellsystems
    3. CUDA
  2. Setting up particles
  3. Running the simulation
    1. Thermostats
      • Langevin
      • LB
      • DPD
      • Isotropic
      • Brownian
      • Stokesian
    2. Integrators
      • Velocity Verlet
      • Isotropic NPT
      • Steepest descent
      • Stokesian Dynamics
    3. Rotational degrees of freedom and particle anisotropy
@jngrad
Copy link
Member Author

jngrad commented Sep 27, 2020

The Brownian integrator is not documented in the user guide.

@jngrad jngrad added this to the Espresso 4.2 milestone Dec 4, 2020
@jngrad jngrad changed the title Dedicated chapters for thermostats and integrators Central chapter for thermostats and integrators Dec 4, 2020
@jngrad jngrad self-assigned this Dec 30, 2020
@kodiakhq kodiakhq bot closed this as completed in #4080 Jan 4, 2021
kodiakhq bot added a commit that referenced this issue Jan 4, 2021
Fixes #3913

Description of changes:
- move thermostats and integrators to the same chapter
- mention the Brownian integrator (links to the Brownian thermostat)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant