[R-pkg-devel] Help with 2 NOTES

[Jaime Salvador]

Hi Ivan,

Thanks for you reply, I appreciate your help.

Actually, with your help, I fixex the issues :


  *   I enclose between single quotes Redata, RXDB and DICX
  *   I fixed the version of binaries downloaded, for example redatamx-core-linux-20241010.zip
  *   The binary DLL/so library generate a LOG file in temp/redatam directory, so I remove the directory in .onUnload
  *   The \dontrun in examples. To run the examples, we need a database, the package doesn't include a sample database, so the code can't be run. I remove \dontrun in functions that can run without a database (for example redatam_version)
  *   All the exported functions, now have the name redatam_XXX (where XXX is open, close, etc.)

Regarding to the DLL downloading during the configurarion, REDATAM is not an open source software (the source code is controlled by CELADE-CEPAL), but the binary is freely  distributed, you can get a copy from https://redatam.org/es/aplicaciones (Redatam X version)

Thanks


-------------------------------------
Jaime Salvador
| Ideas&Bits
| mobile: +593 0987285748
| email: jaime.salvador using ideasybits.com<mailto:jaime.salvador using ideasybits.com>
| site: www.ideasybits.com<https://urldefense.com/v3/__http:/www.ideasybits.com__;!!C3eXzRIY6Mw!YcjWo_ljX1Mppy0kki8QDmz6mHQIrU1XSABeGK31GmB50zdsNenbNNyRCZvpmI0lgbWo$>



------ Mensaje original ------
De "Ivan Krylov" <ikrylov using disroot.org<mailto:ikrylov using disroot.org>>
Para "Jaime Salvador" <jaime.salvador using ideasybits.com<mailto:jaime.salvador using ideasybits.com>>
Cc "r-package-devel using r-project.org" <r-package-devel using r-project.org<mailto:r-package-devel using r-project.org>>
Fecha 10/10/2024 3:30:18
Asunto Re: [R-pkg-devel] Help with 2 NOTES

Dear Jaime Salvador,

Welcome to R-package-devel!

В Wed, 9 Oct 2024 20:42:24 +0000
Jaime Salvador <jaime.salvador using ideasybits.com<mailto:jaime.salvador using ideasybits.com>> пишет:

New submission

This one is expected for new packages.

Possibly misspelled words in DESCRIPTION:
DICX (12:67)
RXDB (12:59)
Redatam (2:34, 12:40)

Wrap the abbreviations and software names into 'single quotes':
https://contributor.r-project.org/cran-cookbook/description_issues.html#formatting-software-names.
Some of the acronyms may need to be explained in the submission
comments in the CRAN web form.

I see you've tried to put the acronyms into the WORDLIST, but I think
the interface for spell-checking the package is different (see
?'aspell-utils') and involves saving vectors of words into .rds files
in .aspell and describing them in .aspell/defaults.R.

(If anything reads WORDLIST, why doesn't grep -r WORDLIST src on the R
source code show anything?)

* checking for detritus in the temp directory ... NOTE

Found the following files/directories:
‘redatam’

Some code from your package that ran during R CMD check (it's usually
examples or tests) created the file named "redatam" in the temporary
directory and didn't remove it afterwards. I'm not seeing any R code do
this, so it must be something in the libredengine DLL. Can you make it
remove its temporary files during _redatam_redatam_destroy()?

There are more serious problems not flagged by R CMD check.

Your code downloads a binary DLL during the configuration stage. What
is the license under which it is shipped? Will it work on macOS? CRAN
policy recommends including sources or downloading (and compiling on
the fly) a fixed version rather than the latest one, although I know
of exceptions for some open-source libraries.

Almost all your examples are wrapped in \dontrun{}. Generally, this is
only allowed for code that absolutely has no chance of running as
written, for example, due to a missing API key:
https://contributor.r-project.org/cran-cookbook/general_issues.html#structuring-of-examples

Your NAMESPACE contains a lot of extraneous stuff. You intend for the
user to call functions redatam.open(), redatam.query(), and so on, but
roxygen2 misinterpreted those as S3 methods and registered them for the
S3 generic redatam, which doesn't do anything and isn't properly
documented. It's best to get rid of the S3 generic and only export the
functions by name (e.g. export(redatam.open, redatam.query)) instead of
resorting to exportPattern("^[[:alpha:]]+"). Maybe it's best to just
avoid dots in the function names altogether, lest something else
misinterprets them as S3 methods.

Your "Description:" field says "you need R (>=4.0)", which is better
expressed as a separate field, "Depends: R (>= 4.0)".

--
Best regards,
Ivan

	[[alternative HTML version deleted]]