On 12/9/22 4:33 AM, Duncan Murdoch wrote:
On 08/12/2022 9:20 p.m., Gabriel Becker wrote:
Hi Spencer,

My 2c.

According to the docs, factors are special-cased. Other S3 'classes' could
be special-cased, such as Date in your example, I suppose, but it is not
clear how what you're describing could be implemented for the general case.

Suppose I define an S3 "class" called my_awesome_class, and have a list of 3 of them in it, and no other guarantees are provided. What should, or even
could, R do in the case of unlist(list_of_awesomes)?

There is no guarantee that I as an S3 developer have provided a c method
for my class such that we could say the unlist call above is equivalent
(roughly) to do.call(c, list_of_awesomes), nor that I provided any other
particular "mash this set of my_awesome_class objects into one". Nor is it
even guaranteed that the concept of combining my_awesome_class objects is
even coherent, or would produce a new my_awesome_class object when
performed if it is.

For the non-recursive case of unlist, do.call(c, list_of_awesomes) is a pretty reasonable expectation.  Wouldn't the simplest change be to make no change to unlist, but suggest this alternative in the documentation?

Duncan Murdoch


Hi, Duncan and Gabrien:


          That's ultimately what I did.  My real problem was more like:


(todaytomorrow <- list(d0=Sys.Date()+0:1, d1=Sys.Date()+2:3))

$d0
[1] "2022-12-09" "2022-12-10"

$d1
[1] "2022-12-11" "2022-12-12"


          I wanted the minimum of the maxima.  So I naively did:


(tt2 <- sapply(todaytomorrow, min))


   d0    d1
19335 19337


          So I next tried:


> (tt3 <- as.Date(tt2))

Error in as.Date.numeric(tt2) : 'origin' must be supplied


I believe that the default "origin" for "as.Date.numeric" should be "1970-01-01". I implemented that several years ago in Ecfun:


> (tt4 <- Ecfun::as.Date1970(tt2))

          d0           d1
"2022-12-09" "2022-12-11"


However, before getting here, I first misdiagnosed the problem with "tt2", believing that "min" not "sapply" was stripping the attributes.


          After fixing that problem, I came to Duncan's solution:


> (tt4 <- lapply(todaytomorrow, min))

$d0
[1] "2022-12-09"

$d1
[1] "2022-12-11"


(maximin <- do.call('max', tt4))
        
[1] "2022-12-11"


Conclusion: It would help to document Duncan's solution using "do.call" and avoiding "unlist" and "sapply". I brought it to the attention of this group, because I wondered if you might want to change the language -- or at least the documentation, as Duncan suggested.


          Thanks,
          Spencer Graves


That said, your example was of length one, we could special case (the
default method of) unlist so that for x *not a list*, we're guaranteed that

identical(unlist(list(x)), x) == TRUE

This would simplify certain code, such as the one from your motivating
example, but at the cost of making the output of unlist across inputs less
consistent and less easy to reason about and predict. In other words the
answer to the question "what class is unlist(list_of_awesomes)? " would
become "it depends on how many of them are in the list"... That wouldn't be
a good thing on balance, imho.

Best,
~G

On Thu, Dec 8, 2022 at 5:44 PM Spencer Graves <
spencer.gra...@effectivedefense.org> wrote:

Consider:


  > str(unlist(list(Sys.Date())))
   num 19334


  > str(unlist(list(factor('a'))))
   Factor w/ 1 level "a": 1


           I naively expected "str(unlist(list(Sys.Date())))" to return an
object of class 'Date'.  After some thought, I felt a need to ask this
list if they think that the core R language might benefit from modifying
the language so "str(unlist(list(Sys.Date())))" was of class 'Date', at
least as an option.


           Comments?
           Thanks,
           Spencer Graves


  > sessionInfo()
R version 4.2.2 (2022-10-31)
Platform: x86_64-apple-darwin17.0 (64-bit)
Running under: macOS Big Sur 11.7.1

Matrix products: default
LAPACK:
/Library/Frameworks/R.framework/Versions/4.2/Resources/lib/libRlapack.dylib

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

loaded via a namespace (and not attached):
[1] compiler_4.2.2 tools_4.2.2

______________________________________________
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel


    [[alternative HTML version deleted]]

______________________________________________
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel

______________________________________________
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel

______________________________________________
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel

Reply via email to