Movatterモバイル変換

[0]ホーム
URL:

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
alexshoe merged 10 commits intoplotly:masterfromdberardi99:bugfix-for-issue-4787
Oct 16, 2025

Conversation

@dberardi99
Copy link
Contributor

Fixplotly/plotly.py#4787

As suggested by@alexcjohnson inplotly/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
@gvwilsongvwilson added communitycommunity contribution P2considered for next cycle featuresomething new labelsOct 24, 2024
@archmojarchmoj added this to thev3.1.0 milestoneOct 24, 2024
@alexcjohnson
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

@gvwilsongvwilson assignedgvwilson and unassignedarchmojNov 21, 2024
@gvwilsongvwilson added the cscustomer success labelNov 21, 2024
@archmoj
Copy link
Contributor

@dberardi99 Could you please resolve the conflicts with master?

@alexshoe
Copy link
Contributor

alexshoe commentedOct 15, 2025•
edited
Loading

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

Copy link
Contributor

@emilyklemilykl left a comment•
edited
Loading

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 indraftlogs/ describing this change.

@alexshoealexshoe merged commit8e3709f intoplotly:masterOct 16, 2025
5 checks passed
@emilykl
Copy link
Contributor

Woohoo, thanks@dberardi99 and@alexshoe ! 🎉

alexshoe reacted with heart emoji

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

Reviewers

@camdecostercamdecostercamdecoster left review comments

@alexcjohnsonalexcjohnsonalexcjohnson left review comments

@emilyklemilyklemilykl approved these changes

Assignees

@alexshoealexshoe

Labels

communitycommunity contributioncscustomer successfeaturesomething newP2considered 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

7 participants

@dberardi99@alexcjohnson@archmoj@alexshoe@emilykl@camdecoster@gvwilson
[8]ページ先頭
©2009-2025 Movatter.jp