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
Add missing docstrings #1234
Add missing docstrings #1234
Conversation
time range or data (see also further ). | ||
|
||
In addition you can provide the following arguments: | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[pep8] reported by reviewdog 🐶
W293 blank line contains whitespace
straxen/analyses/pulse_plots.py
Outdated
:param path: Relative path where the PDF should be stored. By default | ||
it is the directory of the notebook. | ||
:param detector_ending: Ending of the corresponding detector. Empty | ||
string for TPC '_nv' for neutron-veto and '_mv' muon-veto. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[pep8] reported by reviewdog 🐶
W291 trailing whitespace
time range or data (see also further ). | ||
|
||
In addition you can provide the following arguments: | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[pep8] reported by reviewdog 🐶
W293 blank line contains whitespace
straxen/analyses/pulse_plots.py
Outdated
:param path: Relative path where the PDF should be stored. By default | ||
it is the directory of the notebook. | ||
:param detector_ending: Ending of the corresponding detector. Empty | ||
string for TPC '_nv' for neutron-veto and '_mv' muon-veto. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[pep8] reported by reviewdog 🐶
W291 trailing whitespace
time range or data (see also further ). | ||
|
||
In addition you can provide the following arguments: | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[pep8] reported by reviewdog 🐶
W293 blank line contains whitespace
straxen/analyses/pulse_plots.py
Outdated
:param path: Relative path where the PDF should be stored. By default | ||
it is the directory of the notebook. | ||
:param detector_ending: Ending of the corresponding detector. Empty | ||
string for TPC '_nv' for neutron-veto and '_mv' muon-veto. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[pep8] reported by reviewdog 🐶
W291 trailing whitespace
…en into add_doc_string_pulse_plots
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Daniel, these docstrings read pretty clearly to me and I see now issues with merging.
Adds missing doc-stings for some mini-analyses.