search

tomwenseleers / export

R package for streamlined export of graphs and data tables.
192 stars 35 forks source link

graph2ppt() doesn't work1 #27

Closed realreal2020 closed 4 years ago

realreal2020 commented 4 years ago

Thanks for Tomwenseleers job in R packages-export, it is so wonderful that I have spent several hours to install it. however, once graph2ppt() was input where I wanted to save my plot generated by R, this bug would appear.

Error: 'ph_with_vg_at' is defunct. Use 'officer::ph_with' instead. See help("Defunct")

So I seek help here! @tomwenseleers

smouksassi commented 4 years ago

there is a pending PR by the rvg author to fix this and related issue due to the deprecation of several functions form the officer ecosystem

realreal2020 commented 4 years ago

there is a pending PR by the rvg author to fix this and related issue due to the deprecation of several functions form the officer ecosystem

sorry to here that! so all we have to do is wait?

smouksassi commented 4 years ago

You can fork , clone and merge the PR reinstall. Or you can download the zip of the repo. Unzip, Replace the old files with the new as per the PR and re- install using devtools::install()

realreal2020 commented 4 years ago

thxI have downloaded the repo zip. after unzipping, graph2office code was also be replaced by new code ph_with. Then, re-zip and local re-install with an old version 'rvg'. but, warning as"export.rdb corrupted". I was failed again…anyway, thank you so much for your kind-------- 原始邮件 --------发件人： Samer Mouksassi notifications@github.com日期： 2020年3月29日周日 半夜12:43收件人： tomwenseleers/export export@noreply.github.com抄送： realreal2020 zjxvzhenzhen@126.com, Author author@noreply.github.com主 题： Re: [tomwenseleers/export] graph2ppt() doesn't work1 (#27) You can fork , clone and merge the PR reinstall. Or you can download the zip of the repo. Unzip, Replace the old files with the new as per the PR and re- install using devtools::install()

—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.

smouksassi commented 4 years ago

There is a workflow for this explained here: https://stackoverflow.com/questions/6022302/how-to-apply-unmerged-upstream-pull-requests-from-other-forks-into-my-fork

cvanderaa commented 4 years ago

Hello, The PR has been merged ! Sorry for the delay... Export should work again after installing the newest version:

devtools::install_github("tomwenseleers/export")

Let me know if issues persist.

realreal2020 commented 4 years ago

Dear cvanderaa, thanks for your excellent work!-------- 原始邮件 --------发件人： cvanderaa notifications@github.com日期： 2020年4月2日周四 晚上10:06收件人： tomwenseleers/export export@noreply.github.com抄送： realreal2020 zjxvzhenzhen@126.com, Author author@noreply.github.com主 题： Re: [tomwenseleers/export] graph2ppt() doesn't work1 (#27) Hello, The PR has been merged ! Sorry for the delay... Export should work again after installing the newest version: devtools::install_github("tomwenseleers/export") Let me know if issues persist.

—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.

cvanderaa commented 4 years ago

Thank you for using our package! If you encounter a new issue, do not hesitate to open a new issue and tagging me in (@cvanderaa).