Doug Kelkhoff
Doug Kelkhoff
Yeah - that's certainly one concern I have. Representing `ExprList` as `List` means that we need to do some dispatch on the `Obj` enum (or maybe in the future, `trait`)...
Fundamentally I have no issue with testing it out. I think we should be able to go this route even if we kept the current `Obj::Vector(Vector::{Double,Character,..})` as well as `Obj::List(List)`...
The closure style is definitely more in the spirit of what I was imagining. I had been thinking that this might be best solved through a `RepType::Iterator`, which covers both...
Digging up an old issue, but there's a mention of this in issue for [the vector re-write that added the first alt-reps](https://github.com/dgkf/R/issues/2#issue-1729531960).
Yeah - that's also a very good point. This might be a good case for doing some benchmarking. I think both styles have merits. Just to put your explicit example...
Thank you again for another thoughtful write-up! The weekday parsing is definitely underdeveloped. It leans primarily on `strptime` to try to convert yearweeks. I must admit that this isn't a...
Thanks - I agree it looks a bit odd right now. At least when the output supports color, the missing fields are highlighted in red. This is the same as...
Thanks @trevorld, then I think `XX` might be the best option then (eg, `"2020-XX-02"`) since there's some precedence for what it represents. I'd still lean toward using the `NA` `pillar`...
Thanks for taking the time to report this @AmyMikhail! There are a couple things going on here. Just to break down the steps a bit, this breaks down into `impute_time(as.partime(x))`...
> Maybe a simpler solution is to add a `TaskCallback` This is a great idea! It was weird to discover that task callbacks get called while in `browser()`, but maybe...