You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Seems to be best at the file maintainer level so that type is set once and not when the user runs it...but that means we need to come up with an nflverse-level type dictionary
Hey @tanho63, interested in whether you've given this further thought recently?
I ran into the column-naming inconvenience a bunch this week while making some updates to nfl_data_py, and I'm considering transforming them all within the module to standardize the outputs. But on the other hand, I value keeping the API close to nflreadr for the sake of inter-communication between R/Python users in the Discord.
column naming or data types? column naming is difficult because we'd need to either support backcompat or do some sort of warn/deprecation process which wouldn't be fast.
I'd broadly be in favour of at least determining a standardized naming scheme that we could move towards in the future, if you have something to propose there. Would likely belong in a new issue since this specific one is about ensuring consistency of doubles, ints, character etc
Where should this responsibility live?
Seems to be best at the file maintainer level so that type is set once and not when the user runs it...but that means we need to come up with an nflverse-level type dictionary
Relatedly: column naming (player? player_name? full_player_name? name? merge_name) etc
The text was updated successfully, but these errors were encountered: