-
Notifications
You must be signed in to change notification settings - Fork 71
Open
Labels
documentationDocumentation, example tutorials, and websitesDocumentation, example tutorials, and websitesgood first issueGood for newcomersGood for newcomers
Description
This issue consists of three efforts:
- There are some cases where the order or presence of parameters in the docstring is inconsistent with the order of parameters in the function itself, such as https://github.com/jonescompneurolab/hnn-core/blob/master/hnn_core/viz.py#L81-L121 . All docstrings should be updated to be accurate with respect to the argument order in every function.
- For parameters which have a default value, but their default is different than
None, these optional parameters should have, default=<value>added to the end of the parameter's first line in the docstring. As an example, this
color : tuple of float | str
RGBA value to use for plotting. By default, 'k' (black)
should become
color : tuple of float | str, default='k'
RGBA value to use for plotting. By default, 'k' (black)
since this implies both that the parameter is optional, and that it has a specific default value. This is in agreement with https://numpydoc.readthedocs.io/en/latest/format.html#parameters .
- For parameters whose default value is
None, these parameters should have, optionaladded to the end of the parameter's first line in the docstring, in a similar manner to above.
Note that any amount of work for this is appreciated! Contributors should not feel like they have to fix every single instance of this issue before making a PR and contributing changes.
Metadata
Metadata
Assignees
Labels
documentationDocumentation, example tutorials, and websitesDocumentation, example tutorials, and websitesgood first issueGood for newcomersGood for newcomers
Type
Projects
Status
Backlog