Re: [R-pkg-devel] NOTEs - Problems with news in 'NEWS.md'

2022-12-12 Thread Henrik Bengtsson
I've got a check_news() function to test that a NEWS.md can be parsed as expected. See attached file (if that gets dropped in the interwebs, it's also at https://gist.github.com/HenrikBengtsson/2361371be4ae8681b3ee2eb361730990). Call it in the folder where the NEWS.md file is. My $.02 /Henrik O

Re: [R-pkg-devel] NOTEs - Problems with news in 'NEWS.md'

2022-12-12 Thread Max Turgeon
Hi Sanjeev, Requirements change over time, and it's the responsibility of the maintainer to make sure the package keeps up with the new requirements. Things that "worked" back in 2010 may no longer fit these new requirements, and such is the case for the NEWS.md file. It's useful to look at ex

Re: [R-pkg-devel] NOTEs - Problems with news in 'NEWS.md'

2022-12-12 Thread Sanjeev Sariya
Dear Devel and Uwe, Sorry about not providing more details. Please see log file links below: https://win-builder.r-project.org/incoming_pretest/GARCOM_1.2.1_20221212_174734/Windows/00check.log https://win-builder.r-project.org/incoming_pretest/GARCOM_1.2.1_20221212_174734/Debian/00check.log I d

Re: [R-pkg-devel] NOTEs - Problems with news in 'NEWS.md'

2022-12-12 Thread Uwe Ligges
On 12.12.2022 18:05, Sanjeev Sariya wrote: Hi there, I made changes to the CRAN package by adding a file to the inst/ext folder. I mention about this in the NEWS.md file however, when I upload the build package .gz file to cran it fails to pass with error message as: * Problems with news i

Re: [R-pkg-devel] NOTEs - Problems with news in 'NEWS.md'

2022-12-12 Thread Duncan Murdoch
On 12/12/2022 12:05 p.m., Sanjeev Sariya wrote: Hi there, I made changes to the CRAN package by adding a file to the inst/ext folder. I mention about this in the NEWS.md file however, when I upload the build package .gz file to cran it fails to pass with error message as: * Problems with news

[R-pkg-devel] NOTEs - Problems with news in 'NEWS.md'

2022-12-12 Thread Sanjeev Sariya
Hi there, I made changes to the CRAN package by adding a file to the inst/ext folder. I mention about this in the NEWS.md file however, when I upload the build package .gz file to cran it fails to pass with error message as: * Problems with news in 'NEWS.md': No news entries found.* There are

Re: [R-pkg-devel] no visible binding for global variable ‘degree_C’ - CRAN check note

2022-12-12 Thread Alexandre Courtiol
Irucka, If you saw still have the note "no visible binding for global variable ‘degree_C’", it implies that somewhere in your package degree_C is still unquotted... Do you have your code online for us to help you more effectively? ++ On Mon, 12 Dec 2022 at 17:20, EcoC2S - Irucka Embry wrote: > H

Re: [R-pkg-devel] no visible binding for global variable ‘degree_C’ - CRAN check note

2022-12-12 Thread EcoC2S - Irucka Embry
Hi Iñaki and Andrew, I'm sorry, I'll clarify my statement here. set_units(T, "degree_C") still produced the R CMD check note, but it works with regards to setting the unit for the R expression. I replaced every instance of set_units(T, degree_C) with set_units(T, "degree_C") in a single funct

Re: [R-pkg-devel] no visible binding for global variable ‘degree_C’ - CRAN check note

2022-12-12 Thread Iñaki Ucar
On Mon, 12 Dec 2022 at 16:43, EcoC2S - Irucka Embry wrote: > > Hi Andrew, set_units(T, "degree_C") does not work; Sorry, do you mean that the code fails? Or the code works but you still see the NOTE? In this case, it is possible that you didn't replace all the instances of set_units(T, degree_C)

Re: [R-pkg-devel] no visible binding for global variable ‘degree_C’ - CRAN check note

2022-12-12 Thread EcoC2S - Irucka Embry
Hi Alexandre, set_units(T, "degree_C") does not prevent the note. Thank you for your suggestion though. Irucka On 11-12-2022 05:15, Alexandre Courtiol wrote: set_units(T, "degree_C") ## using quotes will prevent the R CMD check note and will still work in this case. On Sun, 11 Dec 2022 at 0

Re: [R-pkg-devel] no visible binding for global variable ‘degree_C’ - CRAN check note

2022-12-12 Thread EcoC2S - Irucka Embry
Hi Andrew, set_units(T, "degree_C") does not work; however, setting degree_C as a global variable does work. Thank you for your suggestion. Irucka On 11-12-2022 00:29, Andrew Simmons wrote: You can declare degree_C as a variable before using set_units(): degree_C <- NULL set_units(T, degree

Re: [R-pkg-devel] How to Fix Debian and Linux Check Errors?

2022-12-12 Thread Alexandre Courtiol
Not sure this is random but all you errors appear on platform that seem to use LC_CTYPE = C.UTF-8 https://cran.r-project.org/web/checks/check_flavors.html#r-release-linux-x86_64 I would start digging in this direction... On Mon, 12 Dec 2022 at 14:13, Ismail Otoakhia wrote: > I received the link

[R-pkg-devel] How to Fix Debian and Linux Check Errors?

2022-12-12 Thread Ismail Otoakhia
I received the link below from CRAN https://cran.r-project.org/web/checks/check_results_ardl.nardl.html I cannot reproduce the error in the example files. Also, I have no idea how to fix the error on the r-devel-linux-x86_64-debian-gcc