On 01.03.2018 18:40, Dason Kurkiewicz wrote:
They're just note NOTES. Although I don't like the names your functions
take because of the possibility to confuse with s3 if they have been in the
package as long as you say I don't see why this would be an issue. Just
send an explanation when you submit and I don't think there should be any
issues.
It is an issue, as not only you but also R will treat these as S3 methods.
As these are not S3 methods, you shoudl ideally
- Give them new names
- Deprecate the old usage (but still allow it), so users get warnings
and get migrate to the new names.
Best,
Uwe Ligges
-Dason Kurkiewicz
On Mar 1, 2018 11:36 AM, "Brian G. Peterson" <br...@braverock.com> wrote:
... bumping this in the hopes that someone has an answer ...
On Fri, 2018-01-26 at 10:35 -0600, Brian G. Peterson wrote:
I'm preparing a new release of PerformanceAnalytics, and as usual
this brings the host of CRAN checks.
I get this note:
* checking S3 generic/method consistency ... NOTE
Found the following apparent S3 methods exported but not registered:
mean.LCL mean.UCL mean.geometric mean.stderr
See section 'Registering S3 methods' in the 'Writing R Extensions'
manual.
these are not S3 methods for 'mean', but rather helper functions that
do other things related to means.
These functions have been in the package for over a decade, are used
in
coursework at a number of universities, and we assume are in user
code.
So it seems that changing them would be disruptive.
Now, all that said, what to do about it?
We could register them as S3method{} , and I think calling them
explicitly will still work, though this seems inelegant.
I can't find any way to tell R that these *aren't* S3 methods ...
Any insight would be greatly appreciated.
Regards,
Brian
______________________________________________
R-package-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel
[[alternative HTML version deleted]]
______________________________________________
R-package-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel
______________________________________________
R-package-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel
