search for: koncevicius

But this might require a more detailed investigation. For example I just noticed that even with the patch `@` is still not listed in .S3_methods_table(). KK. > On Apr 29, 2023, at 4:44 PM, Karolis Koncevi?ius <karolis.koncevicius at gmail.com> wrote: > > Hello Kurt, > > With r84341 it now works on my side. > > Warm regards, > Karolis K. > >> On Apr 29, 2023, at 1:24 PM, Kurt Hornik <Kurt.Hornik at wu.ac.at> wrote: >> >>>>>>> Karolis Koncevi?ius writes: &...

...ight require a more detailed investigation. For example I > just noticed that even with the patch `@` is still not listed in > .S3_methods_table(). Afaict base does not register any methods for @ or @<- ? -k > KK. >> On Apr 29, 2023, at 4:44 PM, Karolis Koncevi?ius <karolis.koncevicius at gmail.com> wrote: >> >> Hello Kurt, >> >> With r84341 it now works on my side. >> >> Warm regards, >> Karolis K. >> >>> On Apr 29, 2023, at 1:24 PM, Kurt Hornik <Kurt.Hornik at wu.ac.at> wrote: >>> >>>>&g...

...>> This error is gone if we adjust tools:::.S3_method_markup_regexp to pass for ?@? by adding ?\\@? to regexp. > >> Please note that this now is not dependant on using or not using roxygen2 >> KK. > >>> On Apr 29, 2023, at 12:53 AM, Karolis Koncevi?ius <karolis.koncevicius at gmail.com> wrote: >>> >>> Thank you for such a quick reply, Gabriel, >>> >>> I am not too familiar with the package tools, so cannot speak too confidently, but below is how I see the issue currently. >>> >>> The issue is not for externa...

...ng `@` as a valid for S3. > This error is gone if we adjust tools:::.S3_method_markup_regexp to pass for ?@? by adding ?\\@? to regexp. > Please note that this now is not dependant on using or not using roxygen2 > KK. >> On Apr 29, 2023, at 12:53 AM, Karolis Koncevi?ius <karolis.koncevicius at gmail.com> wrote: >> >> Thank you for such a quick reply, Gabriel, >> >> I am not too familiar with the package tools, so cannot speak too confidently, but below is how I see the issue currently. >> >> The issue is not for external packages to rely on un...

...egexp and not finding `@` as a valid for S3. This error is gone if we adjust tools:::.S3_method_markup_regexp to pass for ?@? by adding ?\\@? to regexp. Please note that this now is not dependant on using or not using roxygen2 KK. > On Apr 29, 2023, at 12:53 AM, Karolis Koncevi?ius <karolis.koncevicius at gmail.com> wrote: > > Thank you for such a quick reply, Gabriel, > > I am not too familiar with the package tools, so cannot speak too confidently, but below is how I see the issue currently. > > The issue is not for external packages to rely on unexported functions from...

...not know whether R-core feels this is something packages/users should be able to do; both decisions strike me as possible, to be honest, depending on details I don't know and/or am not privy to. > > Best, > ~G > > On Fri, Apr 28, 2023 at 1:49?PM Karolis Koncevi?ius <karolis.koncevicius at gmail.com <mailto:karolis.koncevicius at gmail.com>> wrote: >> This issue might go deeper - I was not successful in passing R CMD checks for the usage files. R CMD check kept showing errors for `@` declarations, even thou they were identical to `$` declarations (which passed fine)...

...o fix or document. I do not know whether R-core feels this is something packages/users should be able to do; both decisions strike me as possible, to be honest, depending on details I don't know and/or am not privy to. Best, ~G On Fri, Apr 28, 2023 at 1:49?PM Karolis Koncevi?ius < karolis.koncevicius at gmail.com> wrote: > This issue might go deeper - I was not successful in passing R CMD checks > for the usage files. R CMD check kept showing errors for `@` declarations, > even thou they were identical to `$` declarations (which passed fine). > > Seems like the usage check fu...

...R CMD check kept showing errors for `@` declarations, even thou they were identical to `$` declarations (which passed fine). Seems like the usage check functions are not prepared for `@` - also in tools:::.S3_method_markup_regexp > On Apr 28, 2023, at 10:34 PM, Karolis Koncevi?ius <karolis.koncevicius at gmail.com> wrote: > > I was building a package that uses the new generic @ and kept having errors with ?roxygen2? documentation. ?roxygen2? generated NAMESPACE added `@.newclass` as a newly exported function, not as a S3method. > > At first I thought this must be a bug in roxyge...

...R. But after some investigation I found that ?roxygen2? is using tools:::.get_internal_S3_generis() to decide if the method should be exported as S3method or not. For some reason ?@<-? is listed in there, but ?@? is not. Am I missing some context, or is this an oversight? Kind regards, Karolis Koncevicius

...agree, this is a good idea and would be very helpful in interactive contexts. I have a draft patch implementing this feature here: https://github.com/r-devel/r-svn/pull/122 (Append ?.patch? to the URL to get a raw patch.) Regards, Tomasz > On Mar 31, 2023, at 2:11 PM, Karolis K <karolis.koncevicius at gmail.com> wrote: > > Hello, > > In the current R-devel @ is S3 generic, so we can do things like - for example - use it to extract matrix rows by name: > > .S3method("@", "mm", function(object, name) object[name,]) > m <- structure(matrix(...

Hello, In the current R-devel @ is S3 generic, so we can do things like - for example - use it to extract matrix rows by name: .S3method("@", "mm", function(object, name) object[name,]) m <- structure(matrix(rnorm(20), ncol=2), dimnames=list(paste0("row", 1:10), paste("col", 1:2)), class="mm") m at row1 However, seems like

Hello, I also want to report 2 missed cases of stringsAsFactors=TRUE in base: 1. grid.expand() still uses hard stringsAsFactors=TRUE in its arguments. 2. as.data.frame.table() also keeps factors after conversion from table. >>>>>> Duncan Murdoch >>>>>> on Sun, 12 Apr 2020 08:57:14 -0400 writes: > > > The NEWS for R 4.0.0 says "R now uses

Thank you for such a quick reply, here are some points that I think might have been missed: > I would state the question the other way : why are NAs integer indices allowed? > In my experience, they are sometimes useful but they often delay the detection of bugs. However, due to backward compatibility, this feature cannot be removed. Adding this feature to character indices would worsen the