Conversation
|
@cpsievert this seems to work at first glance as a drop-in replacement to the JS version, based on spot-checking a few plots and running the test suite. I had trouble getting the visual tests to work, so those are the ones skipped in this test count. I was getting the 22 warnings prior to making the JS version update, so those are not new here. Feel free to adjust the R version and notes for it, or handle that separately. Thanks! |
|
We encountered similar issue in our system and posit is blocking the installation of plotly due to plotly.js CVE. Is there any ETA on this release ? Thanks ! |
|
I plan on taking a look next week. There's a good chance updating will cause seom subtle and easy-to-miss regressions, so it's gonna require a careful look from me. |
|
Thanks @bhogan-mitre, I'll take things from here (see #2471) |
Fixes #2463