LandSciTech / pfocal

Fast parallel convolution in R.
https://landscitech.github.io/pfocal/
Other
2 stars 1 forks source link

Release pfocal 1.0.0 #27

Closed VLucet closed 2 years ago

VLucet commented 2 years ago

First release:

Prepare for release:

Submit to CRAN:

Wait for CRAN...

VLucet commented 2 years ago
VLucet commented 2 years ago

URLs throwing warnings due to repo being private, should resolve itself

   Found the following (possibly) invalid URLs:
     URL: https://github.com/LandSciTech/pfocal/
       From: DESCRIPTION
             man/pfocal-package.Rd
       Status: 404
       Message: Not Found
     URL: https://github.com/LandSciTech/pfocal/actions
       From: README.md
       Status: 404
       Message: Not Found
see24 commented 2 years ago

Need to make the description longer

and rcmd check has this note

> checking installed package size ... NOTE
    installed size is  6.7Mb
    sub-directories of 1Mb or more:
      libs   6.4Mb

I am getting an error when I try to run rhub::check_for_cran().

results <- rhub::check_for_cran(email = "sarah.endicott@ec.gc.ca")
-  Building package
Error: 
Build failed, standard output:

* checking for file 'pfocal/DESCRIPTION' ... OK
* preparing 'pfocal':
* checking DESCRIPTION meta-information ... OK
* cleaning src
* installing the package to build vignettes
      -----------------------------------
* installing *source* package 'pfocal' ...
** using staged installation
** libs
Warning in system(cmd) : 'make' not found
ERROR: compilation failed for package 'pfocal'
* removing 'C:/Users/ENDICO~1/AppData/Local/Temp/RtmpIBwrEd/Rinst2bfc7d613e6/pfocal'
      -----------------------------------
ERROR: package installation failed

Standard error:

Warning in cleanup_pkg(pkgdir, Log) :
  unable to run 'make clean' in 'src'
see24 commented 2 years ago

Need to make the description longer

and rcmd check has this note

> checking installed package size ... NOTE
    installed size is  6.7Mb
    sub-directories of 1Mb or more:
      libs   6.4Mb

Apparently this is common for C++ packages and should be fine https://stackoverflow.com/questions/53819970/r-package-libs-directory-too-large-after-compilation-to-submit-on-cran

VLucet commented 2 years ago

Yup the note about size is not a problem as far as i know. Unsure about the make clean issue, could be a bug on r-hub.

see24 commented 2 years ago

@VLucet Do you think it is related to this? https://developer.r-project.org/Blog/public/2021/12/07/upcoming-changes-in-r-4.2-on-windows/ I am trying to figure out if I should install RTools42. Not sure if that will mess up RTools40 or if it would help at all...

VLucet commented 2 years ago

I dont think it is, it looks like the default on Rhub is still Rtools40, but I admit this is a little above my head. It's happened to me to have stuff pop up on Rhub and then disappear the next time I try. Did you check if the warning is repeated?

see24 commented 2 years ago

Ya I tried it just now with a few different configurations: rhub::check_with_sanitizers, rhub::check(platform = 'windows-x86_64-devel'), rhub::check(platform = 'ubuntu-rchk'), and rhub::check_for_cran and they all had the same error message. devtools::check_win_devel() was fine though. I just re-enabled the Github Actions to see if those work.

VLucet commented 2 years ago

Good call on the GA, lets see.

see24 commented 2 years ago

Looks like it is probably an Rhub issue since the GA R-CMD check worked fine.