Vectorise cr_choose_*() about coloratio HOT 2 OPEN

Hi Petr, it's taken me a while to get round to this. This should now work:

remotes::install_github("matt-dray/coloratio")
coloratio::cr_choose_bw(c("white", "black"))
## [1] "black" "white"

Does that help?

I haven't closed this issue because I haven't yet vectorisedcr_choose_color(). This function is a bit different to cr_choose_bw() because the user can set the length of the return object with n (i.e. they want the n colors with greatest contrast). So, vectorising cr_choose_color() might mean returning a list of the same length as col, with elements that are all of length n. Or maybe n is coerced to 1 when a vector of length 2 more more is supplied to col, i.e. it will only return the single color with greatest contrast. Ideas welcome!

from coloratio.

Hi Matt,

I just stumbled upon a similar situation (labelling a heatmap) in my work and was reminded of this exchange - sorry for not reacting! Many thanks, it does help indeed.

Re cr_choose_color(), returning a list would not make it usable in after_scale(), so with that purpose in mind I would go for the latter option, but then perhaps there are other use cases.

Alternatively, you could have cr_choose_one_color() or allow the user to control the type of output via an argument, but that makes things a bit opaque.

from coloratio.