microsoft / RTVS

R Tools for Visual Studio.
MIT License
390 stars 118 forks source link

export plot as PDF or metafile: crash of visual studio #3151

Closed Ecolio closed 7 years ago

Ecolio commented 7 years ago

Hi,

any plot exported using the save as PDF or the copy as metafile trigger a crash and closing of Visual Studio 2017-02-08 1

MikhailArkhipov commented 7 years ago

Could you please send us logs via R Tools | Feedback | Send Frown. Thanks

Ecolio commented 7 years ago

Hi Mikhail,

thank you for your quick reply, you will find the files in attachment,

all the best,

lio

On 8 February 2017 at 20:41, Mikhail Arkhipov notifications@github.com wrote:

Could you please send us logs via R Tools | Feedback | Send Frown. Thanks

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Microsoft/RTVS/issues/3151#issuecomment-278438924, or mute the thread https://github.com/notifications/unsubscribe-auth/AOoYlPWl5cXX1Q2UIAIN7ZTFJDevbWKsks5rahp1gaJpZM4L7Aot .

--


Lionel Jouvet

Ph.D. Marine Biology M.Sc. - B.Sc. - DUETI - DUT

Max Planck Centre MaxO http://lioneljouvet.wix.com/website ____ http://lioneljouvet.wix.com/website

MikhailArkhipov commented 7 years ago

@huguesv - is this something we've fixed in post 0.5?

Exception Info: Microsoft.R.Host.Client.REvaluationException
   at Microsoft.R.Host.Client.RExpressionEvaluatorExtensions.ThrowOnError(System.String, Microsoft.R.Host.Client.REvaluationResult)
   at Microsoft.R.Host.Client.RExpressionEvaluatorExtensions+<ExecuteAsync>d__3.MoveNext()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(System.Threading.Tasks.Task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(System.Threading.Tasks.Task)
   at Microsoft.R.Components.Plots.Implementation.RPlotManager+<ExportToMetafileAsync>d__42.MoveNext()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(System.Threading.Tasks.Task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(System.Threading.Tasks.Task)
   at Microsoft.R.Components.Plots.Implementation.Commands.PlotDeviceCopyAsMetafileCommand+<InvokeAsync>d__3.MoveNext()
MikhailArkhipov commented 7 years ago

Similar to #2680

MikhailArkhipov commented 7 years ago

Seem to work OK in master (coming public soon)

Ecolio commented 7 years ago

thank you,

looking forward. Got it to work eventually by exporting vectorial plots through code line (eg: svg(filename.svg)/pdf(filename.pdf))

best,

lio

On 9 February 2017 at 19:26, Mikhail Arkhipov notifications@github.com wrote:

Seem to work OK in master (coming public soon)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Microsoft/RTVS/issues/3151#issuecomment-278729087, or mute the thread https://github.com/notifications/unsubscribe-auth/AOoYlEHjXQLvM1fT5pVSeJoObR7pN5QQks5ra1pVgaJpZM4L7Aot .

--


Lionel Jouvet

Ph.D. Marine Biology M.Sc. - B.Sc. - DUETI - DUT

Max Planck Centre MaxO http://lioneljouvet.wix.com/website ____ http://lioneljouvet.wix.com/website

huguesv commented 7 years ago

It does seem to work fine using latest build from master. I used the code from http://www.harding.edu/fmccown/r/ as that looks like what was used when you got the crash.

I've looked at the code and it does catch that REvaluationException (it catches base class RException) in RPlotManager.ExportToMetafileAsync. I'm hopeful that this won't occur in our next release.

MikhailArkhipov commented 7 years ago

Feel free to reopen if you see it in 1.0 RC1+. Thanks.