Since ncdf4 (and emld) both use S3 class systems, it is sufficient to simply declare
setOldClass("ncdf4")
some time prior to using setAs() .
Martin
?On 5/24/19, 6:41 PM, "R-package-devel on behalf of Emmanuel Blondel (GMAIL)" <r-package-devel-bounces at r-project.org on behalf of emmanuel.blondel1 at gmail.com> wrote:
Dear all, I write here as i'm in process to submit a revision of
/geometa/ package to CRAN in which i've enabled some coercing methods
between main metadata object ISOMetadata from geometa, and foreign
metadata objects (from emld / ncdf4 packages)
I've received a "pre-test archived" notification, because of the
folllowing warnings dealing with the coercing:
Warning: in method for 'coerce' with signature '"ISOMetadata","emld"': no definition for classes "ISOMetadata", "emld"
Warning: in method for 'coerce' with signature '"emld","ISOMetadata"': no definition for classes "emld", "ISOMetadata"
Warning: in method for 'coerce' with signature '"ncdf4","ISOMetadata"': no definition for classes "ncdf4", "ISOMetadata"
Detail at:
https://win-builder.r-project.org/incoming_pretest/geometa_0.5-0_20190524_153346/Windows/00check.log
I've found this thread that seems to discuss the same matter:
https://github.com/r-spatial/sf/issues/129 . I'm facing the same
situation where there is no good reason enough to add emld, ncdf4 as
Imports, but rather to keep them as Suggests, as they are only used in
these specific converters. In the doubt, i prefer asking here if there
is some good practice to deal with these warnings, eventually to avoid
it, or if this is considered by CRAN team as false positive.
Thanks a lot for your advice,
Best,
Emmanuel
______________________________________________
R-package-devel at r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel