Remove comment about cmap from voxels docstring#24284
Remove comment about cmap from voxels docstring#24284tacaswell merged 1 commit intomatplotlib:mainfrom
Conversation
voxels does not use cmap at all, and thus does not modify shade in relation to cmap being passed in, this appears to be copy/pasted from methods which _do_ use cmap
|
It does not fall through to the underlying |
|
@tacaswell Poly3DCollection makes no references to cmaps whatsoever, relying on the higher level functions to call the map to get facecolors. In fact the light source shading itself is even done in The data for voxels is fundamentally binary (show this voxel or don't) and so don't map onto a cmap. Granted, the For reference, I noticed this while looking into #24277, where the reporter is requesting pretty much exactly that. its easy enough to work around with the tools provided, and bringing in norms and all of the color-focused arguments may be more effort than we wish. |
|
This is another candidate for |
…284-on-v3.6.x Backport PR #24284 on branch v3.6.x (Remove comment about cmap from voxels docstring)
PR Summary
voxelsdoes not usecmapat all, and thus does not modifyshadein relation tocmapbeing passed in, this appears to be copy/pasted from 3D methods which do usecmap.PR Checklist
Tests and Styling
pytestpasses).flake8-docstringsand runflake8 --docstring-convention=all).Documentation
Release Notes
.. versionadded::directive in the docstring and documented indoc/users/next_whats_new/.. versionchanged::directive in the docstring and documented indoc/api/next_api_changes/next_whats_new/README.rstornext_api_changes/README.rst