ClavelLab / maldipickr

Dereplicate And Cherry-pick Mass Spectrometry Spectra
https://clavellab.github.io/maldipickr/
GNU General Public License v3.0
2 stars 0 forks source link

`reframe()` instead of `summarise()` when dplyr > 1.1.0 #11

Closed cpauvert closed 1 year ago

cpauvert commented 1 year ago

In the example of pick_spectra(), reproduced below:

# 4.2 Pick the spectra from clusters without spectra
#   labelled as `picked_before` (hard masking).
pick_spectra(clusters, metadata, "OD600",
  hard_mask_column = "picked_before"
)

A warning advocating for the replacement of summarise() by reframe() :

#> Warning: Returning more (or less) than 1 row per `summarise()` group was deprecated in
#> dplyr 1.1.0.
#> ℹ Please use `reframe()` instead.
#> ℹ When switching from `summarise()` to `reframe()`, remember that `reframe()`
#>   always returns an ungrouped data frame and adjust accordingly.
#> ℹ The deprecated feature was likely used in the maldipickr package.
#>   Please report the issue at <https://github.com/ClavelLab/maldipickr/issues>.

but current dplyr version used is 1.0.10