laresbernardo / lares

Analytics & Machine Learning R Sidekick
https://laresbernardo.github.io/lares/
233 stars 49 forks source link

dplyr 1.0.8 #37

Closed romainfrancois closed 2 years ago

romainfrancois commented 2 years ago

We're about to release dplyr 1.0.8 and as part of running our revdep checks, we've identified that the released version of lares has this issue:

── After ─────────────────────────────────────────────────────────────────────────────────────────────────────
> checking examples ... ERROR
  Running examples in ‘lares-Ex.R’ failed
  The error most likely occurred in:

  > ### Name: distr
  > ### Title: Compare Variables with their Distributions
  > ### Aliases: distr
  > 
  > ### ** Examples
  > 
  > Sys.unsetenv("LARES_FONT") # Temporal
  > data(dft) # Titanic dataset
  > 
  > # Relation for categorical/categorical values
  > dft %>% distr(Survived, Sex)
  Error in `distr()`: Can't subset `.data` outside of a data mask context.
  Backtrace:
      ▆
   1. ├─dft %>% distr(Survived, Sex)
   2. └─lares::distr(., Survived, Sex)
   3.   ├─<unknown>
   4.   └─rlang:::`$.rlang_fake_data_pronoun`(.data, "value")
   5.     └─rlang:::stop_fake_data_subset(call)
  Execution halted

1 error x | 0 warnings ✓ | 0 notes ✓

it appears however that this is fixed in the dev version.

laresbernardo commented 2 years ago

lares 5.0.3 @ CRAN was submitted yesterday and online now. Thanks @romainfrancois