Skip to content

Sync docs, guides, and example configs with current code - #104

Merged
isayev merged 1 commit into
mainfrom
docs/sync-with-code
Jun 13, 2026
Merged

Sync docs, guides, and example configs with current code#104
isayev merged 1 commit into
mainfrom
docs/sync-with-code

Conversation

@isayev

@isayev isayev commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Audited the active documentation (the docs/source Sphinx site, top-level docs, and example/ notebooks) against the current source after the recent bug-hunt merges, and fixed every divergence found by a four-agent parallel review. Left docs/legacy-v2/ (frozen v2 snapshot) and auto-generated API stubs untouched.

Docs (docs/source)

  • Output SDF property names corrected (E_rel/E_relativeE_rel(kcal/mol); E_tot also shown as E_tot(Hartree)).
  • Removed the non-existent --job-name CLI flag example (it's a config-file field).
  • auto3d validate description trimmed to what it actually does.
  • use_ensemble / AUTO3D_USE_ENSEMBLE documented as deprecated no-ops (they warn), not "removed".
  • Replaced an unverifiable conda install -c conda-forge auto3d with conda env create -f installation.yml.
  • Troubleshooting error strings now match the real messages (GPU-index validation; "Only k OR window needs to be specified").
  • Clarified tauto_k/tauto_window are get_stable_tautomers() arguments.

Top-level

  • SECURITY.md supported-versions table → 3.5.x.
  • CHANGELOG.md gains an [Unreleased] section for the recent fixes.

Example notebooks

  • thermodynamic_calculation: fixed the removed get_mol_idx_t= kwarg → mol_info_func=, refreshed 298 → 298.15 K prose, cleared stale help() output.
  • tautomer_with_userNNP: was non-runnable (empty model cells, bogus optimizing_engine="userNNP") — populated the userNNP wrapper and pass the saved model path.

Small code change (to make the shipped example valid)

  • cli/config_schema: expose batchsize_atoms and allow_tf32 in CLIConfig and forward to Auto3DOptions. The shipped parameters.yaml previously failed to load via the documented auto3d run -c path (extra=forbid) and used an invalid window: False; both fixed, with tests.

Fast gate: 649 passed (+2 new CLIConfig tests). All edited .rst parse cleanly.

Audited the active docs (docs/source, top-level, example notebooks) against the
current source and fixed the divergences:

Docs (docs/source):
- usage/quickstart: output SDF property names corrected (E_rel/E_relative ->
  E_rel(kcal/mol); E_tot also exposed as E_tot(Hartree)).
- usage: replace the non-existent `--job-name` CLI flag example with a config-file one.
- cli: `auto3d validate` description trimmed to what it actually checks (format +
  RDKit parse), dropping the unimplemented element/engine check.
- migration: `use_ensemble` / AUTO3D_USE_ENSEMBLE described as deprecated no-op
  (they warn) rather than "removed".
- howto/quickstart: replace the unverifiable `conda install -c conda-forge auto3d`
  with the documented `conda env create -f installation.yml`.
- howto/troubleshooting: error strings match the real messages (GPU index
  validation; "Only k OR window needs to be specified").
- howto/drug_discovery: clarify tauto_k/tauto_window are get_stable_tautomers()
  arguments (not Auto3DOptions fields) and document the _Name grouping key.

Top-level:
- SECURITY.md supported-versions table updated to 3.5.x.
- CHANGELOG.md: add an [Unreleased] section for the recent fixes.

Example notebooks:
- thermodynamic_calculation: fix the removed `get_mol_idx_t=` kwarg ->
  `mol_info_func=`, refresh 298 -> 298.15 K prose, clear stale help() output.
- tautomer_with_userNNP: was non-runnable (empty model cells, bogus
  optimizing_engine="userNNP"); populate the userNNP wrapper, save it, and pass
  the saved model path.

Code (to make the shipped example config valid via the documented CLI):
- cli/config_schema: expose `batchsize_atoms` and `allow_tf32` in CLIConfig and
  forward them to Auto3DOptions; parameters.yaml previously failed to load via
  `auto3d run -c` (extra=forbid) and used an invalid `window: False`.
@isayev
isayev merged commit 551e4b2 into main Jun 13, 2026
8 of 9 checks passed
@isayev
isayev deleted the docs/sync-with-code branch June 13, 2026 04:02
isayev added a commit that referenced this pull request Jul 11, 2026
Sync docs, guides, and example configs with current code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant