NREL / BuildingMOTIF

Building Metadata OnTology Interoperability Framework (BuildingMOTIF). For models, see https://models.open223.info/
https://buildingmotif.readthedocs.io/
Other
49 stars 7 forks source link

cleanup docstrings for public release #174

Closed MatthewSteen closed 1 year ago

MatthewSteen commented 1 year ago

Close #147. Cleans up docstrings from public (mostly) functions and methods. We can cleanup private ones later if we decide to autodoc them. Here are the conventions I tried to follow.

  1. Docstring line length = 79 per PEP8.
  2. All docstrings start on the same line as the opening """ for consistency between one-line and multi-line docstrings and because VS Code's autoDocstring extension populates them this way (PEP257 doesn't specify for multi-line docstrings).
  3. The docstring is a phrase ending in a period per PEP257.
  4. Multi-line docstrings have a one-line summary sentence, followed by a blank line per PEP257.
  5. Parameter docstrings are not sentences, i.e. are mostly lower case without a period.
  6. Omit quotes around forward referenced classes in rtype, which is how autoDocstring extension does it.
  7. Move any TODOs from inside docstrings to body of def.