nflverse / nflfastR

A Set of Functions to Efficiently Scrape NFL Play by Play Data
https://www.nflfastr.com/
Other
425 stars 52 forks source link

calculate_player_stats throws error on name_receiver #387

Closed jbpickthree closed 2 years ago

jbpickthree commented 2 years ago
  1. Have you installed the latest development version of the package(s) in question?

Yes

  1. Describe the bug

When trying to run the calculate_player_stats function, an error is returned regarding the column 'name_receiver'

Error: ! Assigned data 0 must be compatible with existing data. i Error occurred for column name_receiver. x Can't convert double to character.

  1. Reprex

calculate_player_stats(load_pbp(2022), weekly = T)
  1. Expected Behaviour

Typically, as it has been up to this point this season and all of last season since I've used this R package, stats would be calculated successfully and I could continue on with the rest of my code.

  1. nflverse_sitrep()
sitrep ```r -- System Info --------------------------------------------------------------------------------------------------------------------------------------------------------------- * R version 4.1.2 (2021-11-01) * Running under: Windows 10 x64 (build 19044) -- nflverse Packages --------------------------------------------------------------------------------------------------------------------------------------------------------- * nflreadr (1.3.1) * nflseedR (1.1.0) * nflplotR (1.1.0) * nflfastR (4.4.0) * nfl4th (1.0.2) * nflverse (1.0.2) -- nflverse Options ---------------------------------------------------------------------------------------------------------------------------------------------------------- No options set for nflreadr, nflfastR, nflseedR, nfl4th, nflplotR, and nflverse -- nflverse Dependencies ----------------------------------------------------------------------------------------------------------------------------------------------------- * askpass (1.1) * gtable (0.3.1) * progressr (0.11.0) * bit (4.0.4) * hms (1.1.2) * proto (1.0.0) * bit64 (4.0.5) * httr (1.4.4) * purrr (0.3.5) * cachem (1.0.6) * isoband (0.2.6) * R6 (2.5.1) * cli (3.4.1) * janitor (2.1.0) * rappdirs (0.3.3) * clipr (0.8.0) * jsonlite (1.8.2) * RColorBrewer (1.1-3) * codetools (0.2-18) * labeling (0.4.2) * Rcpp (1.0.9) * colorspace (2.0-3) * lattice (0.20-45) * readr (2.1.3) * cpp11 (0.4.2) * lifecycle (1.0.3) * rlang (1.0.6) * crayon (1.5.2) * listenv (0.8.0) * rstudioapi (0.14) * curl (4.3.3) * lubridate (1.8.0) * scales (1.2.1) * data.table (1.14.2) * magick (2.7.3) * snakecase (0.11.0) * digest (0.6.29) * magrittr (2.0.3) * stringi (1.7.6) * dplyr (1.0.10) * MASS (7.3-58.1) * stringr (1.4.1) * ellipsis (0.3.2) * Matrix (1.5-1) * sys (3.4) * fansi (1.0.3) * memoise (2.0.1) * tibble (3.1.8) * farver (2.1.1) * mgcv (1.8-40) * tidyr (1.2.1) * fastmap (1.1.0) * mime (0.12) * tidyselect (1.1.2) * fastrmodels (1.0.2) * munsell (0.5.0) * tzdb (0.3.0) * furrr (0.3.1) * nlme (3.1-159) * utf8 (1.2.2) * future (1.28.0) * openssl (2.0.3) * vctrs (0.4.2) * generics (0.1.3) * parallelly (1.32.1) * viridisLite (0.4.1) * ggplot2 (3.3.6) * pillar (1.8.1) * vroom (1.6.0) * globals (0.16.1) * pkgconfig (2.0.3) * withr (2.5.0) * glue (1.6.2) * prettyunits (1.1.1) * xgboost (1.6.0.1) * gsubfn (0.7) * progress (1.2.2) ```
  1. Screenshots

  2. Additional context

tanho63 commented 2 years ago
-- System Info ---------------------------------------------------------------------------------------------------------------------------------------------------------------
* R version 4.1.2 (2021-11-01)   * Running under: Windows 10 x64 (build 19044)
-- nflverse Packages ---------------------------------------------------------------------------------------------------------------------------------------------------------
* nflreadr (1.3.1)  * nflseedR (1.1.0)  * nflplotR (1.1.0)  
* nflfastR (4.4.0)  * nfl4th   (1.0.2)  * nflverse (1.0.2)  

The current development version of nflfastR is 4.4.0.9011 - can you please install that and try again? See the readme for more details

jbpickthree commented 2 years ago

That seemed to work. My bad. I updated it earlier but it must not have completed successfully. Thanks for the quick reply.