Use the new export to pdf function in plotly

[talgalili]

Use the new orca() function

source: plotly/plotly.R#1089

[jalalsiddiqui]

Thanks. Is it possible to use orca() to download the images for free or do we still have to pay for them?

[jonmmease]

Hi @jalalsiddiqui and @talgalili, orca is open source (MIT license), works offline, and totally free 🙂

[alanocallaghan]

Note: need to install orca (https://github.com/plotly/orca#installation) - the Ubuntu /usr/bin/orca is a screen reader.

Also need recent version of plotly as previous versions had a bug relating to mapbox API tokens.

[talgalili]

I've just tested file="temp.pdf" with the recent heatmaply 0.15.3 version on my computer (after updating to the latest plotly) and it crashed my rstudio (in Windows)).
I didn't install orca. It might be worth adding a check to see if orca is installed, and if not, to send the user to the doc: https://github.com/plotly/orca#installation
@alanocallaghan , does it work fine on your device?

[alanocallaghan]

There's already a check for that in plotly, though as noted /usr/bin/orca is a a screen reader on Ubuntu

[talgalili]

I saw installation instructions for Windows though. In any case, I think it might be better to keep the old export option for now, and use a switch if the user wants to save pdf to move to orca (you can also add an option for the export engine, but I suspect most people wouldn't need it, as they could just use the function directly on the output object, if they want that level of control) What do you think?

…

On Thu, Dec 6, 2018 at 12:45 PM Alanocallaghan ***@***.***> wrote: There's already a check for that in plotly, though as noted /usr/bin/orca is a a screen reader on Ubuntu — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#183 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA7khjF1XV9BA7PFZgFVfVxHcfEXxr6Eks5u2PU4gaJpZM4VIOhE> .

[alanocallaghan]

Running on Windows, I get a warning telling me to install orca, which is expected behaviour. No crashes

For Ubuntu, see: plotly/plotly.R#1426

I would not push this to CRAN yet, but I think switching to orca is the right move.

[alanocallaghan]

I pushed a fallback to export, but I think it's not quite right as we're basically duplicating the deprecation warning in plotly