Skip to content

BUG: range is wrong if layout.axis.type == 'log' #1561

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.

Sign up for GitHub

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

Jump to bottom
Closed
destradafilm opened this issue Apr 6, 2017 · 5 comments
Closed

BUG: range is wrong if layout.axis.type == 'log' #1561

destradafilm opened this issue Apr 6, 2017 · 5 comments

Comments

@destradafilm
Copy link

plotlyjs_bug_range numbers in log

As you can see, the range I'm getting for the y axis
min: plotlyDiv._fullLayout.yaxis.range[0]
max: plotlyDiv._fullLayout.yaxis.range[1]

Is wrong when the axis.type is 'log' unless there's a conversion I'm not aware of

codepen:
http://codepen.io/destrada/pen/qrGJwq
@destradafilm
Copy link
Author

destradafilm commented Apr 7, 2017 •
edited
Loading

Nevermind, I just realized this is a log10 conversion of the number range

@etpinard
Copy link
Contributor

etpinard commented Apr 7, 2017

@destradafilm I was just going to write that.

That log10 conversion is unfortunate, but there's no way for us to fix it in a backward-compatible way.

This is essentially the same issue as #1258 that we're planing on fixing in v2. Thanks for posting.

@etpinard etpinard closed this as completed Apr 7, 2017
@oricou
Copy link

oricou commented Sep 4, 2018 •
edited
Loading

It seems I use Plotly v. 2.7 and it still has the bug.

% pip show plotly 
Name: plotly
Version: 2.7.0

@alexcjohnson
Copy link
Collaborator

@oricou that’s v2 of plotly.py. plotly.js is still on v1.

@oricou
Copy link

oricou commented Sep 5, 2018

ok thanks. Any date for version 2? I couldn't find any roadmap...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@alexcjohnson @destradafilm @etpinard @oricou