Kirill Müller comments

Results 2418 comments of


                                            Kirill Müller

Does duckdb_constraints() contain all relevant information?

Currently, `BoundForeignKeyConstraint` doesn't contain schema information. I wonder if we should move towards fulfilling the assertion in https://github.com/krlmlr/duckdb/commit/e7c0e134e5411acaea5fbca82ae46e01046d6bd3 .

test_all_types function integration

For the R connector, the following columns need translations in 192e3cf8ab0a42a0cbaee69a639ebdaaaeee54dc: - `timestamp_tz` - `time_tz` (what is this?) - `uuid` (to ALTREP string with class?) - `interval` (to R's `difftime`)...

use ... to specify path in i_am()?

Thanks for raising this. I think these days `/` works as a file separator everywhere? What's the motivation behind splitting the path?

use_here()

Thanks for your contribution, it looks very useful. A few suggestions: - Expand `T` and `F` - Replace `1:x` with `seq_len()` - Replace `sapply()` with `vapply()` or `map_chr()` - Run...

International?

Thanks. What do you see with `here()` ?

International?

Is this on Windows? Can you please try `here(enc2utf8(...))` ?

here often fails to identify directory on google drive on windows

Thanks. What's the output of `here::dr_here()` ?

Revisit injection properties of `!!` with grouped data frames

Ideally, `!!` and `.env$` would behave identically for all practical purposes. Is this the only exception?

Add dplyr_sticky_cols() or similar

Would stickyness imply focus (#6252)?

Conditionally mutate selected rows

We can fake it already, but overwriting would be a tad faster: ``` r library(tidyverse) df # A tibble: 5 x 1 #> a #> #> 1 1 #> 2...