Skip to content
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

[BUG] changing axis type (linear <->logarithmic) doesn't change on chart.update() #4939

Closed
ldemailly opened this issue Nov 10, 2017 · 2 comments
Closed

[BUG] changing axis type (linear <->logarithmic) doesn't change on chart.update() #4939

ldemailly opened this issue Nov 10, 2017 · 2 comments
Labels
type: bug
Milestone
Version 2.7.2

Comments

@ldemailly
Copy link

ldemailly commented Nov 10, 2017 •
edited
Loading

Expected Behavior

changing the type and update() should change the display type

Current Behavior

stays linear

Possible Solution

workaround is to rewrite the dataset with a different axis id but that seems overkill (also I then have 2 axis showing)

Steps to Reproduce (for bugs)

https://codepen.io/anon/pen/GOWBpx

  1. click the link
  2. axis label changes, axis type does not

Context

I need to switch axis type to help visualize home drawn (x,y) histograms

Environment
@ldemailly
Copy link
Author

ldemailly commented Nov 11, 2017 •
edited
Loading

I actually can't seem to find any workaround
reverse direction: https://codepen.io/anon/pen/dZvjLK

@ldemailly ldemailly mentioned this issue Nov 11, 2017
Merged
@ldemailly
Copy link
Author

Fixed in #4198

ldemailly added a commit to fortio/fortio that referenced this issue Nov 11, 2017
@ldemailly
Allow setting of log scale on either axis
a1555f2 
(Needed to patch in
chartjs/Chart.js#4198
In order to get chartjs/Chart.js#4939
Axis type switching to work)
@ldemailly ldemailly mentioned this issue Nov 11, 2017
Merged
ldemailly added a commit to fortio/fortio that referenced this issue Nov 13, 2017
@ldemailly
Ability to fix X axis min,max and switch to log on both axis (#17)
2a4a06a 
* Put the url being tested on the page

* Allow setting of log scale on either axis

(Needed to patch in
chartjs/Chart.js#4198
In order to get chartjs/Chart.js#4939
Axis type switching to work)

* Allow the setting of min on X axis too

Ignore 0 for min when in log

* Smaller chart.js with all the fixes

0 log is ok now
@benmccann benmccann added this to the Version 2.8 milestone Jan 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug
Projects
None yet
Milestone
Version 2.7.2
Development

No branches or pull requests
3 participants
@benmccann @ldemailly @etimberg