BugFix: `place_acker` output should be 2D array #1181

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.

Sign up for GitHub

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

Jump to bottom

Open

guptavaibhav0 wants to merge 1 commit into python-control:main from guptavaibhav0:main

Contributor

guptavaibhav0 commented Sep 9, 2025

place_acker function's output is a 1D array, while it should be a 2D array as described in its doc string.


          BugFix: place_acker output should be 2D matrix

37ccb27

coveralls commented Sep 9, 2025

coverage: 94.734%. remained the same
when pulling 37ccb27 on guptavaibhav0:main
into abeb0e4 on python-control:main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet