V3 (old) of d3.js Required

[lmorgadodacosta]

It seems that you need V3 of d3.js (now at version 5), since some of the functions used to render dmrs have been renamed.

https://stackoverflow.com/questions/38391411/what-is-the-d3-js-v4-0-equivalent-for-d3-scale-category10/38391511#38391511

[goodmami]

Yes, I have not been able to keep delphin-viz up to date with D3, but luckily V3 is still available, and that is what we pull in for the demo:

delphin-viz/demo/index.html

Line 15 in 30f6144

<script src="https://d3js.org/d3.v3.min.js" charset="utf-8"></script>

I think at a minimum we should mention the version requirement on the README, but it would be great if someone could refresh the visualization function to use V5 of d3.js (or, even better, harmonize all visualizers to the same framework, whether it's D3 or something else).

[ned2]

OK, so step one, we need to document the version requirements.

I agree it would be nice to get all the components using the same framework (and I think D3 is likely to be the right choice). This is something I could work on, but I'm fairly time-limited at the moment, so that could be a while off.

@goodmami, one thing I had been wondering about is the synchronisation of your DMRS visualizer code. It lives somewhere else, and we just have a dump of a particular version right? Might there be a more sensible way of doing this?

[goodmami]

@ned2 the visualizer code here was copied from Demophin, where it still is kept separate for no reason other than I haven't updated Demophin... but I don't really plan to; I think it should be deprecated in favor of delphin-viz. Therefore, the delphin-viz version of dmrs.js is now the canonical version.

[ned2]

Great, good to know! That certainly keeps things simpler too.

…

On Sun, 23 Sep 2018 at 12:29, Michael Wayne Goodman < ***@***.***> wrote: @ned2 <https://github.com/ned2> the visualizer code here was copied from Demophin, where it still is kept separate for no reason other than I haven't updated Demophin... but I don't really plan to; I think it should be deprecated in favor of delphin-viz. Therefore, the delphin-viz version of dmrs.js is now the canonical version. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#5 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACs1GNzWCzaI8nFXSBebZGp3oWN_HBUzks5udvILgaJpZM4W1hah> .

-- nedned.net