Skip to content

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

Merged
alexshoe merged 10 commits into
plotly:masterfrom
dberardi99:bugfix-for-issue-4787
Oct 16, 2025
Merged

Add "SI extended" formatting rule for the tick exponents#7249
alexshoe merged 10 commits into
plotly:masterfrom
dberardi99:bugfix-for-issue-4787

Conversation

@dberardi99

Copy link
Copy Markdown
Contributor

Fixplotly/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.

Comment threadsrc/plots/cartesian/axes.js Outdated
Comment threadsrc/plots/cartesian/layout_attributes.js
@gvwilsongvwilson added community community contribution P2 considered for next cycle feature something new labels Oct 24, 2024
@archmojarchmoj added this to the v3.1.0 milestone Oct 24, 2024
@alexcjohnson

Copy link
Copy Markdown
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

@gvwilsongvwilson assigned gvwilson and unassigned archmojNov 21, 2024
@gvwilsongvwilson added the cs customer success label Nov 21, 2024
@archmoj

Copy link
Copy Markdown
Contributor

@dberardi99 Could you please resolve the conflicts with master?

@alexshoe
alexshoeforce-pushed the bugfix-for-issue-4787 branch from fe8e096 to 15fa0baCompareOctober 15, 2025 18:15
@alexshoe

alexshoe commented Oct 15, 2025

Copy link
Copy Markdown
Contributor

@emilykl@camdecoster The changes here LGTM. Fixed the merge conflict in src/plots/cartesian/axes.js.

Comment threadsrc/plots/cartesian/axes.js
Comment threadsrc/plots/cartesian/axes.js Outdated

@emilyklemilykl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Left a few suggestions for improving the clarity of the code, but LGTM!

@dberardi99 Please add a new draftlog file in draftlogs/ describing this change.

Comment threadsrc/plots/cartesian/axes.js Outdated
Comment threadsrc/plots/cartesian/axes.js Outdated
Comment threadsrc/plots/cartesian/axes.js Outdated
Comment threadsrc/plots/cartesian/axes.js Outdated
@alexshoe
alexshoe merged commit 8e3709f into plotly:masterOct 16, 2025
5 checks passed
@emilykl

Copy link
Copy Markdown
Contributor

Woohoo, thanks @dberardi99 and @alexshoe ! 🎉

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

Labels

communitycommunity contributioncscustomer successfeaturesomething newP2considered for next cycle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SI prefixes don't display Peta (P) correctly

7 participants

@dberardi99@alexcjohnson@archmoj@alexshoe@emilykl@camdecoster@gvwilson