-
Notifications
You must be signed in to change notification settings - Fork 5
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
Code documentation #21
Comments
@felixbur, |
ok, cool. Sometimes the blog postings are outdated, I would go through the code and add comments to the classes and main functions what they do, and what are the arguments and return values, so that the code could be used outside nkululeko |
@felixbur If you have time during ICASSP 2023, is it possible to make a small hackathon for nkululeko, partially on Documentation part? I will come to that conference wholly and will attend the Show and Tell session of Nkululeko. |
oh, hi, cool, we can definetly meet! |
I just realized that you already activated GitHub pages for this repo. Actually, we can use that for the documentation but I think using readthedocs is better as you pointed out above in realpython link.
|
I would suggest to use Google conventions for docstrings. def from_db(y: float) -> float:
"""Value converted from decibel scale.
Args:
y: amplitude value in decibels
Returns:
amplitude value
Examples:
>>> from_db(0)
1.0
>>> from_db(-3)
0.7079457843841379
"""
return np.power(10, y / 20) |
Reference to that proposal: https://google.github.io/styleguide/pyguide.html |
The code is almost entierly undocumented.
https://realpython.com/documenting-python-code/
The text was updated successfully, but these errors were encountered: