-
Notifications
You must be signed in to change notification settings - Fork 35
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade d3 or use different graphing library for display (not capture) #180
Comments
Ahhh, i see-- we upgraded d3 to So, that issue is blocking us... |
Noting this was caught by @sidntrivedi012 in #77 (comment) and it's actually d3's dependency of an old version of Here is where d3 upgraded to a newer |
Oh, that's not right, that's d3 |
OK, then here, they went to JSDOM 7/8: d3/d3#2596 |
And here, they mention specifically the d3 v3.4 and v3.5 threshold: d3/d3#2578 (comment) and |
We could try requiring [email protected] ... |
OK, first version of d3 to make JSDOM optional is actually |
Refactoring and using a new graphing library would be a lot of work, because we use d3 in the calibration routines and they are pixel-level CSS tuned with multiple complex drag interactions -- see https://publiclab.org/wiki/spectral-workbench-calibration#Video
However, we have identified a problem with d3
v3.5.17
and have had to revert tov3.4.0
. The error is:The revert is in #175, after initially bumping in #165.
Perhaps we could stop using
nvd3
(https://nvd3.org/) which is not compatible with modernd3
(v5 for example and maybe v4), and then could get d3 up to a more recent version.But we should really start by moving the calibration routine out of the Ruby on Rails SpectralWorkbench.org project and into this repo. The script for calibration UI (as in the video linked at top) is:
https://github.com/publiclab/spectral-workbench/blob/98dbde7a881788f7bd48db59eee616d6de70054b/app/assets/javascripts/analyze.js#L52-L60
https://github.com/publiclab/spectral-workbench/blob/98dbde7a881788f7bd48db59eee616d6de70054b/app/assets/javascripts/analyze.js#L122-L143
Lot of work here, but worthwhile!
The text was updated successfully, but these errors were encountered: