-
-
Notifications
You must be signed in to change notification settings - Fork 163
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
Complete LRE user guide #2542
Complete LRE user guide #2542
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2542 +/- ##
=======================================
Coverage 98.72% 98.72%
=======================================
Files 92 92
Lines 4161 4167 +6
=======================================
+ Hits 4108 4114 +6
Misses 53 53 ☔ View full report in Codecov by Sentry. |
6fa56af
to
57bd891
Compare
For some reason, the RTD build is taking longer than 45-50 mins. |
@natestemen This PR is still in draft status due to part 4 of the user guide. I pinged you so that you could review what's already in this PR. |
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.
Mostly minor comments, but overall looks really great!
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.
Looking good! This is going to be great addition when it's fully incorporated.
def _get_scale_factor_vectors( | ||
def get_scale_factor_vectors( |
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.
Reminder to myself that I will need to convert this function to allow for arbitrary QPROGRAM
.
Co-authored-by: nate stemen <[email protected]>
Co-authored-by: nate stemen <[email protected]>
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.
LGTM! Thanks Purva!
Will merge in the morning and do the release once this is in :)
Description
fixes #2371
Merge after #2541
License
Before opening the PR, please ensure you have completed the following where appropriate.