Skip to content

Update axis_labels guidance for 0.9 - #1117

Open
TimMonko wants to merge 4 commits into
napari:mainfrom
TimMonko:doc/axis-labels
Open

Update axis_labels guidance for 0.9#1117
TimMonko wants to merge 4 commits into
napari:mainfrom
TimMonko:doc/axis-labels

Conversation

@TimMonko

Copy link
Copy Markdown
Member

References and relevant issues

Follow-ups to napari/napari#9282 and napari/napari#9293 (and others)

Description

Softly updates the guides to reflect the new axis labels inheritance behavior. For now, I didn't totally remove the ability to set it directly on the dims, but I think we are working towards that? I'm not sure how strongly we want to do that yet, given the popup renaming exists as does the API.

Especially happy to hear folks ideas on how best to go about this, but I mostly wanted to be 0.9 ready.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Aug 19, 2026
@TimMonko TimMonko added this to the 0.9.0 milestone Aug 20, 2026
TimMonko added a commit that referenced this pull request Aug 21, 2026
# References and relevant issues

Found this while working through #1117 and realized also relveant to
#1113

# Description

Due to the changes with box Xarray and axis_labels inheritance, this
guide can be cleaned up quite a bit,
which is nice because then it focuses on the actual handedness info!

I opted for a new PR since this touches both related things and it would
be easier to review this way? I hope :laugh:
Comment thread docs/guides/handedness.md
sst.sst,
name='sea surface temp',
colormap='magma',
axis_labels=sst.sst.dims,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is unnecessary because of the xarray inheritance, right?

@jni jni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

All good, but minor query about the update to handedness.md?

@jni

jni commented Aug 24, 2026

Copy link
Copy Markdown
Member

@Czaki if you want to tag before @TimMonko wakes up, feel free to merge this: my concern in this PR is not critical to fix before release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants