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
drawstyle accepts anything as default rather than raising #6143
Comments
That seems like a reasonable course of action. In the (near) future the plan is to use traitlets everywhere which will help with this sort of thing, but documenting what needs to be validated by code that validates them in probably a good thing. |
Most importantly, if you add tests it will make sure the traitlets do not cause any regressions. |
I just wasn't sure if there was additional parsing being done as for |
Added a check that forbids `drawstyle` keywords that are not listed in the documentation.
Added a check that forbids `drawstyle` keywords that are not listed in the documentation. Added tests to mimic linestyle tests. Added self to mailmap.
Setup
matplotlib.__version__
= 1.5.1matplotlib.__version__numpy__
= 1.6Python 3.5.1, IPython 4.1.2 running under anaconda
OS: Red Hat 6.5
Matplotlib installed with
conda install matplotlib
.Issue
Supplying invalid
drawstyle
parameter toplot
(and possible others) does not raise an error. I would expect the following not to work, but it does:For contrast, trying the same thing with
fillstyle
raises the expected error:Notes
The issue seems to be happening here:
matplotlib/lib/matplotlib/lines.py
Line 988 in 3ba9279
would mess up something or another. If not, I would be glad to make the change.
To help us understand and resolve your issue please check that you have provided
the information below.
The text was updated successfully, but these errors were encountered: