Skip to content

V0.7 lqr objective bugfix #67

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

Merged
merged 2 commits into from
Apr 4, 2022
Merged

V0.7 lqr objective bugfix #67

merged 2 commits into from
Apr 4, 2022

Conversation

jbwillis
Copy link
Contributor

Resolves #66 by using qf for final cost. Adds tests that catch original error and pass with fix.

@jbwillis jbwillis added the bug Something isn't working label Mar 30, 2022
@jbwillis jbwillis requested a review from bjack205 March 30, 2022 13:59
@jbwillis jbwillis self-assigned this Mar 30, 2022
Copy link
Member

@bjack205 bjack205 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks for catching this bug!

@bjack205 bjack205 merged commit 2ce2008 into v0.7 Apr 4, 2022
@bjack205 bjack205 deleted the v0.7_LQRObjective_bugfix branch April 4, 2022 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy