Skip to content

POC: DOC/Gallery example "Ternary diagram": Migrate frame settings to the new Frame/Axis syntax#4505

Draft
yvonnefroehlich wants to merge 2 commits intomainfrom
update-frame-ternary
Draft

POC: DOC/Gallery example "Ternary diagram": Migrate frame settings to the new Frame/Axis syntax#4505
yvonnefroehlich wants to merge 2 commits intomainfrom
update-frame-ternary

Conversation

@yvonnefroehlich
Copy link
Copy Markdown
Member

@yvonnefroehlich yvonnefroehlich commented Apr 7, 2026

Description of proposed changes

Update the gallery example basemaps/ternary.py to use the newly implemented Frame and Axis parameter classes.
Need aaxis, baxis, caxis similar to xaxis, yaxis, zaxis.

Fixes partly #4494

Preview:

Guidelines

Slash Commands

You can write slash commands (/command) in the first line of a comment to perform
specific operations. Supported slash command is:

  • /format: automatically format and lint the code

@yvonnefroehlich yvonnefroehlich added this to the 0.19.0 milestone Apr 7, 2026
@yvonnefroehlich yvonnefroehlich self-assigned this Apr 7, 2026
@yvonnefroehlich yvonnefroehlich added documentation Improvements or additions to documentation skip-changelog Skip adding Pull Request to changelog labels Apr 7, 2026
@yvonnefroehlich yvonnefroehlich marked this pull request as draft April 7, 2026 13:17
"cafg+lAir component+u %",
],
frame=Frame(
aaxis=Axis(
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.

Hmmm. This one is interesting. I don't want to add aaxis/baxis/caxis to Frame, but what else can do we?

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.

What about let users still pass xaxis/yaxis/zaxis, and we internally convert them to -Ba/-Bb/-Bc?

Copy link
Copy Markdown
Member

@Chuan1937 Chuan1937 Apr 9, 2026

Choose a reason for hiding this comment

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

Modify:#4517

@yvonnefroehlich yvonnefroehlich changed the title POC: DOC/Gallery example "ternary": Migrate frame settings to the new Frame/Axis syntax POC: DOC/Gallery example "Ternary": Migrate frame settings to the new Frame/Axis syntax Apr 8, 2026
@yvonnefroehlich yvonnefroehlich changed the title POC: DOC/Gallery example "Ternary": Migrate frame settings to the new Frame/Axis syntax POC: DOC/Gallery example "Ternary diagram": Migrate frame settings to the new Frame/Axis syntax Apr 8, 2026
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 skip-changelog Skip adding Pull Request to changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants