-
Notifications
You must be signed in to change notification settings - Fork 52
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
Degradation #436
Degradation #436
Conversation
…id_request' doc strings
Gunna have to get those tests fixed before we do much more with this I think |
Hi, yes definitely, got pulled away by a more pressing issue with the runs i need this for, will fix asap. |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #436 +/- ##
==========================================
+ Coverage 87.02% 87.09% +0.06%
==========================================
Files 122 122
Lines 17047 17120 +73
==========================================
+ Hits 14836 14911 +75
+ Misses 2211 2209 -2
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.
Nice job! A few minor requests, with special emphasis on the formatting thorughout.
Huge kudos for doing all the validation checks AND including tests. You're basically a reV developer now :)
Now it is failing on docstring lines that are over the line length limit, though these were there before and it had passed. Not sure how you'd like to handle this. |
Ooops! Those are totally my bad. Not much to do about it though, since we need that length for the link to work (I'm pretty sure?). Anyways, I disabled the check for those docstrings |
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.
Nice job!
This adds the ability to get lifetime outputs, though with restrictions on which variables may be returned. Definitely open to suggestion about how best to handle the limited available lifetime outputs from SAM.