8000 Handle scalar `timeresp` in `solve_flat_ocp` by roryyorke · Pull Request #1111 · python-control/python-control · GitHub
[go: up one dir, main page]

Skip to content

Handle scalar timeresp in solve_flat_ocp #1111

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 3 commits into from
Feb 1, 2025

Conversation

roryyorke
Copy link
Contributor

Fixes gh-1110.

@coveralls
Copy link
coveralls commented Feb 1, 2025

Coverage Status

coverage: 94.659% (+0.01%) from 94.649%
when pulling a05b9d5 on roryyorke:gh1110-solve_flat_ocp
into ebff125 on python-control:main.

Comment on lines 473 to 475
assert np.max(abs(r1.x-r2.x)) == 0
assert np.max(abs(r1.u-r2.u)) == 0
assert np.max(abs(r1.y-r2.y)) == 0
Copy link
Contributor
@bnavigator bnavigator Feb 1, 2025

Choose a reason for hiding this comment

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

Is this a fancy way to say r1.x == r2.x while employing some opaque shape casting?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

just laziness! fixed.

@roryyorke
Copy link
Contributor Author

the failures are during install optional dependencies phase, before tests are run, and must be unrelated.

@murrayrm
Copy link
Member
murrayrm commented Feb 1, 2025

Reran the tests and that fixed the problem.

@murrayrm murrayrm merged commit 2cb0520 into python-control:main Feb 1, 2025
23 checks passed
@murrayrm murrayrm added this to the 0.10.2 milestone Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

solve_flat_ocp and scalar timepts
4 participants
0