-
Notifications
You must be signed in to change notification settings - Fork 4
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
First Pass Documentation (docstrings) For Latent Folder #73
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #73 +/- ##
==========================================
+ Coverage 90.28% 90.45% +0.17%
==========================================
Files 26 28 +2
Lines 535 545 +10
==========================================
+ Hits 483 493 +10
Misses 52 52
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Overall it looks good. Ping me when the comments/discussions are addressed
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.
All looks good, @AFg6K7h4fhy2. I took care of a few things were requested and created an issue for another (#79). Will merge this and continue with the others.
This pull request includes a first pass
./model/src/pyrenew/latent
./model/src/pyrenew/latent
Note: You will find some TODO placeholders, mostly in the Notes section of some docstrings. These occur where the author was uncertain about standard procedure or the content.