Fix plotting warnings in Pytest, add news for removal of six support#215
Conversation
There was a problem hiding this comment.
Warning 1: UserWarning: linestyle is redundantly defined by the 'linestyle' keyword argument
properties already has linestyle shown below:
properties.update({"color": color, "linestyle": lineStyle, "linewidth": lineWidth})
| _, properties = self.__translateStyles(style) | ||
| curveRef = self.subplot.plot(xData, yData, **properties)[0] | ||
| if "legend" in style: | ||
| self.subplot.legend(**legendBoxProperties()) |
There was a problem hiding this comment.
Warning 2: No artists with labels found to put in legend. Note that artists whose label start with an underscore are ignored when legend() is called with no argument.
Solution - just check whether style contains legend like it was checked in line 321 of extendedplotframe.py
if "legend" in style:
properties["label"] = style["legend"]
return stylestr, properties
bobleesj
left a comment
bobleesj
left a comment
There was a problem hiding this comment.
Ready for review @sbillinge
sbillinge
left a comment
sbillinge
left a comment
There was a problem hiding this comment.
This looks good. I will merge, but @bobleesj it would be good if we can figure out a way to test that this hasn't broken any thing in the interface. Can we maybe figure out where these code statements are used and make sure the GUI hasn't changed?
2 participants
Closes #213
Closes #152