Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] import_schedule shows wrong week for rest #86

Closed
2 tasks done
darkhark opened this issue Apr 1, 2024 · 5 comments
Closed
2 tasks done

[BUG] import_schedule shows wrong week for rest #86

darkhark opened this issue Apr 1, 2024 · 5 comments

Comments

@darkhark
Copy link

darkhark commented Apr 1, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Have you installed the latest development version of the package(s) in question?

  • I have installed the latest development version of the package.

If this is a data issue, have you tried clearing your nflverse cache?

I am not using an nflverse R package (nflreadr, nflfastR) to access this data.

What version of the package do you have?

0.3.1

Describe the bug

If the home_rest and away_rest columns are meant to represent each team's bye weeks, they appear to default to 7. This is for week 1:
image
The rest value changes depending on the week for the same teams too.

Reprex

import nfl_data_py as nfl
nfl.import_schedules([2023])[['away_team', 'home_team', 'away_rest', 'home_rest']]

Expected Behavior

The values in rest week show the same bye week for the same team no matter the week.

nflverse_sitrep

NA

Screenshots

Added to description

Additional context

Not urgent, but I planned on imputing values using the bye week.

@smenon02
Copy link

smenon02 commented Apr 2, 2024

is there anyone working on this? If not, I can take it up

@darkhark
Copy link
Author

darkhark commented Apr 2, 2024

is there anyone working on this? If not, I can take it up

You're the first reply. Feel free to go for it!

@nateb7722
Copy link

These columns look like they represent the days of rest for each team going into that game. not the bye week. Is that not the intended use of these columns? Should I be calculating days rest differently?

@tanho63
Copy link
Member

tanho63 commented Sep 19, 2024

image
https://nflreadr.nflverse.com/articles/dictionary_schedules.html

this is indeed days of rest for home/away team (i.e. number of days since last game before the game date)

@darkhark
Copy link
Author

Awesome, thank you for the clarification!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants