Skip to content
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

Returning the Poly3DCollection when calling bar3d #4494

Merged
merged 1 commit into from
Jun 4, 2015
Merged

Returning the Poly3DCollection when calling bar3d #4494

merged 1 commit into from
Jun 4, 2015

Conversation

jeremyfix
Copy link
Contributor

bar3d does not return the Poly3DCollection which is annoying if one wishes to update the bar plot;

Without the modification, I feel I have to do :
ax.bar3d( ...)
col = ax.collections[-1]
And then for animating :

While(True):
col.remove()
ax.bar3d(...)
col = ax.collections[-1]

with the modification, I can merge the ax.bar3d(..); col = .. ; as col = ax.bar3d(...)

bar3d does not return the Poly3DCollection which is annoying if one wishes to update the bar plot;

Without the modification, I feel I have to do :
ax.bar3d( ...)
col = ax.collections[-1]
And then for animating :

While(True):
   col.remove()
   ax.bar3d(...)
   col = ax.collections[-1]

with the modification, I can merge the ax.bar3d(..); col = .. ;   as  col = ax.bar3d(...)
tacaswell added a commit that referenced this pull request Jun 4, 2015
FIX/API: Returning the Poly3DCollection when calling bar3d
@tacaswell tacaswell merged commit 6fe1225 into matplotlib:master Jun 4, 2015
@tacaswell
Copy link
Member

@jeremyfix Good catch!

@WeatherGod backport to color_overhaul?

@WeatherGod
Copy link
Member

oh, wow. Yeah, backport. I bet it used to be there but then got lopped off at some point.

tacaswell added a commit that referenced this pull request Jun 5, 2015
FIX/API: Returning the Poly3DCollection when calling bar3d
@tacaswell
Copy link
Member

tacaswell commented Jun 5, 2015

cherry-picked as 141d485

@jeremyfix jeremyfix deleted the patch-1 branch June 5, 2015 04:37
@QuLogic QuLogic added this to the v1.5.0 milestone Oct 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants