Skip to content

Slicer inspector - #552

Merged
nvaytet merged 9 commits into
mainfrom
inspector-slicer
Apr 14, 2026
Merged

Slicer inspector#552
nvaytet merged 9 commits into
mainfrom
inspector-slicer

Conversation

@nvaytet

@nvaytet nvaytet commented Apr 11, 2026

Copy link
Copy Markdown
Member

Combining the range slicer with the inspector plot:

%matplotlib widget
import plopp as pp
from plopp.data.examples import clusters3d

N = 256
da = clusters3d(nclusters=500, seed=12).hist(z=N, y=N, x=N)
pp.inspector(da, dim='z', mode='rectangle', logc=True)
Screenshot_20260411_104410

Fixes scipp/ess#15

@nvaytet
nvaytet changed the base branch from main to speedup-rectangle-inspector April 11, 2026 08:47
Base automatically changed from speedup-rectangle-inspector to main April 13, 2026 14:07
@nvaytet
nvaytet marked this pull request as ready for review April 13, 2026 14:07
nvaytet
nvaytet commented Apr 13, 2026
View reviewed changes
Comment thread docs/plotting/inspector-plot.ipynb
"outputs": [],
"source": [
"p"
"This can be done using `continuous_update=False` in the argument list."

@nvaytet nvaytet Apr 13, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need to render yet another figure just to illustrate the continuous_update arg. We reduce a little docs size bloat.

@jl-wynen

jl-wynen commented Apr 14, 2026

Copy link
Copy Markdown
Member

pp.inspector(da, dim='z', mode='rectangle', logc=True)

You set logc=True in your example, should this also set norm=log for the 1D plot? Those to axes show the same quantity, right?

@nvaytet

nvaytet commented Apr 14, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

See discussion in #513 (comment)

It's always up for debate, but the current behaviour is intentional 😉

@nvaytet
nvaytet merged commit 7424484 into main Apr 14, 2026
6 checks passed
@nvaytet
nvaytet deleted the inspector-slicer branch April 14, 2026 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jl-wynen jl-wynen jl-wynen approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Feature Request: Adding slicer to the inspector plot.

2 participants

@nvaytet @jl-wynen