Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add "SI extended" formatting rule for the tick exponents #7249

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

Open
dberardi99 wants to merge 3 commits into plotly:master
base: master
Choose a base branch
Loading
from dberardi99:bugfix-for-issue-4787

Conversation

Copy link

@dberardi99 dberardi99 commented Oct 24, 2024

Fix plotly/plotly.py#4787

As suggested by @alexcjohnson in plotly/plotly.py#4787 (comment), a new "SI extended" tick exponents' formatting rule has been added to display numbers in the full SI unit range (10^-30 to 10^30).

In addition, the 20.json mock and its baseline have been modified to show how this new mode works.

alexcjohnson reacted with heart emoji
@gvwilson gvwilson added community community contribution P2 considered for next cycle feature something new labels Oct 24, 2024
@archmoj archmoj added this to the v3.1.0 milestone Oct 24, 2024
Copy link
Collaborator

LGTM, thanks @dberardi99! It’ll need a draftlog entry, I’d call it an addition rather than a bug fix. @archmoj I see you have this slotted for the next release after 3.0, sounds reasonable, I’ll defer to you for a final review

dberardi99 reacted with heart emoji

@gvwilson gvwilson assigned gvwilson and unassigned archmoj Nov 21, 2024
@gvwilson gvwilson added the cs customer success label Nov 21, 2024
Copy link
Contributor

archmoj commented Jul 18, 2025

@dberardi99 Could you please resolve the conflicts with master?

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

@alexcjohnson alexcjohnson alexcjohnson left review comments

At least 1 approving review is required to merge this pull request.

Labels
community community contribution cs customer success feature something new P2 considered for next cycle
Projects
None yet
Milestone
v3.1.0
Development

Successfully merging this pull request may close these issues.

SI prefixes don't display Peta (P) correctly

AltStyle によって変換されたページ (->オリジナル) /