Skip to content

Commit

Permalink
Merge pull request #172 from OpenFreeEnergy/docs-cleanup
Browse files Browse the repository at this point in the history
Docs cleanup
  • Loading branch information
richardjgowers committed Apr 10, 2023
2 parents aa537fc + 6b59681 commit b001339
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 13 deletions.
10 changes: 5 additions & 5 deletions gufe/chemicalsystem.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,14 +18,14 @@ class ChemicalSystem(GufeTokenizable, abc.Mapping):
components
The molecular representation of the chemical state, including
connectivity and coordinates. This is a frozendict with user-defined
labels as keys, `Component`s as values.
labels as keys, :class:`.Component`\ s as values.
box_vectors
Numpy array indicating shape and size of unit cell for the system. May
be a partial definition to allow for variability on certain dimensions.
name
Optional identifier for the chemical state; used as part of the
(hashable) graph node itself when the chemical state is added to an
`AlchemicalNetwork`.
:class:`.AlchemicalNetwork`.
"""

Expand All @@ -42,15 +42,15 @@ def __init__(
components
The molecular representation of the chemical state, including
connectivity and coordinates. Given as a dict with user-defined
labels as keys, `Component`s as values.
labels as keys, :class:`.Component`\ s as values.
box_vectors
Optional `numpy` array indicating shape and size of unit cell for
Optional ``numpy`` array indicating shape and size of unit cell for
the system. May be a partial definition to allow for variability on
certain dimensions.
name
Optional identifier for the chemical state; included with the other
attributes as part of the (hashable) graph node itself when the
chemical state is added to an `AlchemicalNetwork`.
chemical state is added to an :class:`.AlchemicalNetwork`.
"""
super().__init__()
Expand Down
11 changes: 8 additions & 3 deletions gufe/network.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,14 +11,19 @@


class AlchemicalNetwork(GufeTokenizable):
"""A network of `ChemicalSystem`s as nodes, `Transformation`s as edges.
"""A network with all the information needed for a simulation campaign.
Nodes are :class:`.ChemicalSystem`\ s and edges are
:class:`.Transformation`\ s.
Attributes
----------
edges : FrozenSet[Transformation]
The edges of the network, given as a `frozenset` of `Transformation`s.
The edges of the network, given as a ``frozenset`` of
:class:`.Transformation`\ s.
nodes : FrozenSet[ChemicalSystem]
The nodes of the network, given as a `frozenset` of `ChemicalSystem`s.
The nodes of the network, given as a ``frozenset`` of
:class:`.ChemicalSystem` \ s.
name : Optional identifier for the network.
"""
Expand Down
10 changes: 5 additions & 5 deletions gufe/transformations/transformation.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,26 +15,26 @@
class Transformation(GufeTokenizable):
"""An edge of an alchemical network.
Connects two `ChemicalSystem`s, with directionality.
Connects two :class:`.ChemicalSystem`\ s, with directionality.
Attributes
----------
stateA : ChemicalSystem
The starting `ChemicalSystem` for the transformation.
The starting :class:`.ChemicalSystem` for the transformation.
stateB : ChemicalSystem
The ending `ChemicalSystem` for the transformation.
The ending :class:`.ChemicalSystem` for the transformation.
protocol : Protocol
The protocol used to perform the transformation.
Includes all details needed to perform required
simulations/calculations and encodes the alchemical pathway used.
May also correspond to an experimental result.
mapping : Optional[Dict[str, ComponentMapping]]
Mapping of e.g. atoms between the `stateA` and `stateB`
`ChemicalSystem`s.
:class:`.ChemicalSystem`\ s.
name : Optional[str]
Optional identifier for the transformation; set this to a unique value
if adding multiple, otherwise identical transformations to the same
`AlchemicalNetwork` to avoid deduplication
:class:`.AlchemicalNetwork` to avoid deduplication
"""

Expand Down

0 comments on commit b001339

Please sign in to comment.