You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am able to reproduce the reported behavior of missing zero markers. My hypothesis is some previous version of Matplotlib included marker edges by default, so facecolors='none' was sufficient to cause circle outlines. One way to achieve it now is
regarding the TypeError exception, I was not able to produce the error using the tip of master branch, whereas I can reproduce the error using the previous release version 0.7.0.
Activity
murrayrm commentedon Aug 8, 2017
Can you provide a code snippet that generates both errors? Thanks.
elviseno commentedon Aug 9, 2017
Hi Murray,
The control package is 0.7.0.
A simple test code used,
It reports,
control.matlab.pzmap(tf) is working fine except that zeros are not shown.
In pzmap file,
The zeros are shown after removing facecolors setting.
It seems the 2 pzmap functions are essentially the same one. Sorry I'm not coder and can't digg too much.
Thanks very much.
slivingston commentedon Aug 9, 2017
so, completing the example from above to include necessary imports and the
showcommand, we haveslivingston commentedon Aug 9, 2017
I am able to reproduce the reported behavior of missing zero markers. My hypothesis is some previous version of Matplotlib included marker edges by default, so
facecolors='none'was sufficient to cause circle outlines. One way to achieve it now isslivingston commentedon Aug 9, 2017
regarding the
TypeErrorexception, I was not able to produce the error using the tip ofmasterbranch, whereas I can reproduce the error using the previous release version 0.7.0.slivingston commentedon Aug 16, 2017
@Fankbai context?
Fankbai commentedon Aug 26, 2017
@slivingston
Sup3rGeo commentedon Feb 22, 2018
PR #193 addresses it
murrayrm commentedon Dec 28, 2018
Fixed in PR #193.