Dragoș Moldovan-Grünfeld

Results 25 comments of Dragoș Moldovan-Grünfeld

This fails at: https://github.com/hadley/adv-r/blob/dc49c3872c3530ac08716fd4f4c235b01266a4ce/Control-flow.Rmd#L106 given that the latest version of R (4.2) errors now if the condition has length > 1.

> Have you considered adding a registration step? A registration step would definitely simplify this. There was had a brief conversation (some of which is captured in the [capstone doc](https://docs.google.com/document/d/1OlVjZhMHY_X9lyZNw9LrBANvW_OgzNiQwXMjqIPShV0/edit?disco=AAAAbHEJGPE)...

> Thanks for this! One comment, so far. It would be nice to have an idea of which parts of this are working, and which parts still need work before...

@paleolimbot I like your suggestion in https://github.com/apache/arrow/pull/13789#issuecomment-1214992856. Even if we don't require the user to self-register, that function can really simplify my current logic (as we can use it instead...

**Update**: * user-defined functions work with `mutate()` and `filter()` * we can translate nested function definitions Give both of these work and the approach is rather complex, by walking the...

> > Give both of these work and the approach is rather complex, by walking the AST, I will now go more in the direction suggested by @paleolimbot (in [#13789...

The CI failure (Windows & R 3.6) seems to be related to https://github.com/apache/arrow/pull/13661.

Hi Garrick, {rsthemes} is really cool. Thanks for all the work you put into it. My go-to themes are the Solarized (both Dark and Light). I really like your changes...

@gadenbuie thanks a lot. Wow, that was quick!

I am experiencing something similar on an Intel Mac: ```r > pak::pkg_install("broom") → Will update 1 package. → Will download 1 CRAN package (1.80 MB). + broom 0.7.10 → 0.7.11...