tomwenseleers / export

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

Export was removed from CRAN. #41

Closed lincj1994 closed 1 year ago

lincj1994 commented 3 years ago

Hi, TOM. Package 'export' was removed from cran. I wonder if you can upload it again. Thanks. Lin.

cvanderaa commented 3 years ago

Hi @lincj1994 ,

Indeed, export was removed from CRAN because some changes in other packages broke our package. This is now fixed and we were planning to do it but I suddenly forgot about it... I will put export back to CRAN as soon as possible!

agilebean commented 1 year ago

Hi @tomwenseleers, any chance you can update your awesome package and bring it back to CRAN any time soon? I think it's the best in the R world, so I really hope for an update...

tomwenseleers commented 1 year ago

I would love to, but the CRAN check gives me some warning, which is the reason CRAN won't accept it as it is (only for Debian, not for Mac or Windows, and don't have that installed here; I only have Ubuntu here, where it doesn't give any warnings). So a bit tricky for me to resolve that - so unfortunately I don't quite know how I can get it accepted by CRAN or how I can resolve that issue.... If anyone would be able to figure that out I would be happy to accept a PR though! Or @cvanderaa - would you know what we need to do to get our package to pass the CRAN tests and get it back on CRAN? It would be nice indeed to have it back up there...

cvanderaa commented 1 year ago

I can have a look at it this weekend. I'll try to set up a GitHub Action workflow to run the checks on different OS.

cvanderaa commented 1 year ago

Hi @tomwenseleers, I set up the GHA for the different OS. I used a generic workflow for standard R check from the rlib-actions repo. Beside installing xquartz for macOS and setting RGL params, I didn't have to change the code of export and all checks passed. So, I believe the package is ready to be sent to CRAN :tada:

tomwenseleers commented 1 year ago

Ha that's great @cvanderaa! I just submitted it, let's see what they say! Hope it's all OK now! Many thanks for your help!

tomwenseleers commented 1 year ago

Hey @cvanderaa - I just got the package back, saying "Flavor: r-devel-linux-x86_64-debian-gcc Check: for detritus in the temp directory, Result: NOTE Found the following files/directories: 'calibre_6.9.0_tmp_9r6xkapd' 'calibre_6.9.0_tmp__rujrznq' 'calibre_6.9.0_tmp_dozypqjs' 'calibre_6.9.0_tmp_fp924pjj' 'calibre_6.9.0_tmp_l3azsdjz' 'calibre_6.9.0_tmp_pfmb2i96' 'calibre_6.9.0_tmp_vnqazp87' 'calibre_6.9.0_tmp_xcpm861s'

which suggests you open a web browser in non interactive mode. Please always protect such call by if(interactive()).

Please fix and resubmit."

Do you have any thought how we have to fix that?

cvanderaa commented 1 year ago

Ah, I had such files popping up locally as well, I thought I could ignore them. I'll try debugging this and keep you updated!

tomwenseleers commented 1 year ago

If I try it on https://builder.r-hub.io/, selecting Debian under advanced it passes btw, https://builder.r-hub.io/status/export_0.3.0.tar.gz-1b2b07da115fd81bc89112bf2adbda8d. But presumably it doesn't check for those temporary files there... I think it may also be because they use a noninteractive session there I believe...

cvanderaa commented 1 year ago

I added if (interactive()) to a few examples that opened my browser when running R CMD check . locally. I hope this solves the issue :crossed_fingers: I'll need some more time to see how we could run the examples with GHA in a non-interactive mode, so that these errors pop up on our side in a reproducible and automated fashion.

tomwenseleers commented 1 year ago

Thanks Chris! Just uploaded it, and it passes the pre-checks OK now, so fingers crossed! Thanks again for the help!

tomwenseleers commented 1 year ago

Still had to fix something minor in the documentation, but now CRAN informed me it's back online: https://cran.r-project.org/web/packages/export/index.html! Yahoo!