DOC: switch to O-O interface in basic examples #7066
Conversation
| for color in ['red', 'green', 'blue']: | ||
| n = 750 | ||
| x, y = rand(2, n) | ||
| scale = 200.0 * rand(n) | ||
| plt.scatter(x, y, c=color, s=scale, label=color, | ||
| ax.scatter(x, y, c=color, s=scale, label=color, | ||
| alpha=0.3, edgecolors='none') |
There was a problem hiding this comment.
Need to adjust indent for alpha?
There was a problem hiding this comment.
@dopplershift Ahh yes. Good catch. I'll wait for the CI to wrap up before pushing again.
| plt.xlabel('Performance') | ||
| plt.title('How fast do you want to go today?') | ||
| ax.barh(y_pos, performance, xerr=error, align='center', | ||
| color='green', ecolor='black') |
There was a problem hiding this comment.
There seems to be an indentation problem here.
There was a problem hiding this comment.
oh geez -- that's embarrassing.
|
Apart from my comments, this looks good! |
f8b02f6 to
a9fec83
Compare
|
whoops -- had to restore the labels on the |
| plt.yticks(y_pos, people) | ||
| plt.xlabel('Performance') | ||
| plt.title('How fast do you want to go today?') | ||
| ax.barh(y_pos, performance, xerr=error, align='center', |
There was a problem hiding this comment.
If we do not want to backport this to v2.x, this can use @story645 's categorical work. ax.barh(people, performance) should 'just work'
There was a problem hiding this comment.
I am in favor of merging now and backporting, and create another PR for the categorical bar plot.
DOC: switch to O-O interface in basic examples Conflicts: examples/lines_bars_and_markers/barh_demo.py whitespace vs new code
|
Thanks! backported to v2.x as 215fbd1 |
Small PR to limit the calls to
pltin the lines, bars, and markers example.Took some artistic liberties:
ax.plotfor this examplecolorandecolorI think that about covers it.