Solve #862 and #864: bode_plot phase wrapping incorrect for multiple systems by henklaak · Pull Request #863 · python-control/python-control
Navigation Menu
{{ message }}
python-control / python-control Public
- Notifications You must be signed in to change notification settings
- Fork 453
Merged
Solve #862 and #864: bode_plot phase wrapping incorrect for multiple systems#863
murrayrm merged 5 commits intopython-control:mainfrom
henklaak:main
Solve #862 and #864: bode_plot phase wrapping incorrect for multiple systems#863
murrayrm merged 5 commits intopython-control:mainfrom
henklaak:main
Conversation
Copy link Copy Markdown
Contributor
henklaak
commented
Feb 17, 2023
henklaak
commented
Function argument clobbered by local assignment. Use local variable instead.
henklaak added 4 commits
February 17, 2023 13:58
Copy link Copy Markdown
Contributor Author
matplotlib 3.6.3 => 3.7.* seems to break our tests. Will investigate further.
Copy link Copy Markdown
Contributor Author
henklaak
commented
Feb 17, 2023
henklaak commented
Feb 17, 2023
henklaak
changed the title
Solve #862: bode_plot phase wrapping incorrect for multiple systems
Solve #862 and #864: bode_plot phase wrapping incorrect for multiple systems
Copy link Copy Markdown
coveralls
commented
Feb 17, 2023
coveralls commented
Feb 17, 2023
Copy link Copy Markdown
Member
murrayrm
commented
Feb 18, 2023
murrayrm commented
Feb 18, 2023Thanks for the fixes!
murrayrm
merged commit
346bc40
into
python-control:main
This was referenced
Feb 18, 2023
murrayrm
added this to the
0.9.4 milestone
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants