rickshaw icon indicating copy to clipboard operation
rickshaw copied to clipboard

Support for d3 5.x

Open niemyjski opened this issue 9 years ago • 11 comments

What is the support for d3 5.x like? Any known issues?

niemyjski avatar Jul 02 '16 14:07 niemyjski

Has anyone ran into issues with this?

niemyjski avatar Oct 06 '16 15:10 niemyjski

Yep, I did. Run stacked chart and faced d3.layout.stack() error because d3.layout got undefined. (read somewhere it was split into a separate file for d3 4.x) Reverted back to latest 3.x branch and it solved my issue.

guillaumegarcia13 avatar Jan 12 '17 22:01 guillaumegarcia13

I'm interested about this, I'm new to d3.js and the examples talk about d3.layout and I can't find this file. Even if there is a "recent" 1.6 version of rickshaw, nothing looks updated on the website...

Flaburgan avatar Jan 29 '17 16:01 Flaburgan

I wish a core contributor would respond, the lack of response kind of tells me this project is dead :(. I wish I was better at charting/visuals and I had more time, then I would contribute in the time in which I don't have :(

niemyjski avatar Feb 03 '17 15:02 niemyjski

I'm comprehensively updating the code and examples to work with d3 v4; my team uses Rickshaw and we can't keep postponing the d3 version upgrade. I'll push my code to GitHub this weekend, as soon as I clean up my commit history for public consumption.

Would anyone @shutterstock be interested in merging a PR to update to d3 v4? I can see from #517 that y'all have largely stepped back from maintaining this. I'm happy to send a PR with my code and to help keep the code healthy if you want contributions. Or if you'd rather keep this repo stable, I'm also happy to keep my fork running for my team (and anyone else who would prefer to use that).

butterflyhug avatar Mar 31 '17 19:03 butterflyhug

@butterflyhug any updates on this?

niemyjski avatar May 02 '17 15:05 niemyjski

I've got code on my fork. Note that my current default branch is d3v4 -- not master.

I've added a section at the top of the readme to note a couple minor issues that I noticed from testing my code with the provided examples. I've also tested with the existing unit tests and my team's own codebase, both of which seem to be working flawlessly. I'm sure that there are a few features and/or configurations that haven't been covered by any of these testing methods, and those things may be broken.

I'm still happy to send a PR with my changes if @shutterstock is interested. In the meantime, I'm happy to take PRs to address remaining issues on my fork. If we get to a point where there are no known issues and I still haven't heard any official interest in supporting D3 v4, then I'll think about our options for independently publishing releases and etc.

butterflyhug avatar May 03 '17 22:05 butterflyhug

@butterflyhug:

Would anyone @shutterstock be interested in merging a PR to update to d3 v4? I can see from #517 that y'all have largely stepped back from maintaining this. I'm happy to send a PR with my code and to help keep the code healthy if you want contributions. Or if you'd rather keep this repo stable, I'm also happy to keep my fork running for my team (and anyone else who would prefer to use that)

Yes, I would be more than happy to look at a PR for this update, and I would love to have help maintaining this repository. PR away!

I'm still happy to send a PR with my changes if @shutterstock is interested.

:wave:

RichardLitt avatar Aug 25 '17 19:08 RichardLitt

Any fix should include replacing deprecated calls to d3.scale.linear() with d3.scaleLinear()

socketbox avatar Dec 18 '17 23:12 socketbox

What's missing here? Response from @shutterstock?

Justus-Maier avatar Mar 16 '18 09:03 Justus-Maier

Updated title and issue to 5.x since we are now two majors behind.

niemyjski avatar Mar 16 '18 13:03 niemyjski