dynastyprocess / data

An open-data fantasy football repository, maintained by DynastyProcess.com
https://dynastyprocess.com
GNU General Public License v3.0
73 stars 19 forks source link

Adding player IDs from nflscrapR, nflfastR, and Pro Football Focus #15

Closed TheMathNinja closed 3 years ago

TheMathNinja commented 4 years ago

I'm loving the map of player IDs in the mfl_players() function. Wondering if a few ID columns could get added/mapped:

nflscrapR / (I think this is old NFL API). Example: Kyle Orton = 00-0023541 nflfastR / (new NFL API). Example: Kirk Cousins = 32013030-2d30-3032-3936-303492e9d55e

You can find their code easily on GitHub

TheMathNinja commented 4 years ago

Also, Pro Football Reference and Pro Football Focus IDs would be extremely helpful. I know you can check out a map of these here:

https://github.com/leesharpe/nfldata/blob/master/data/pff_pfr_map_v1.csv

tanho63 commented 4 years ago

Probably won't hit it from the actual code, but coming at it from here should be okay:

https://github.com/guga31bb/nflfastR-data/tree/master/roster-data

tanho63 commented 4 years ago

PFR I've worked loosely on in the past, and mapping those to PFF is an interesting idea too.

TheMathNinja commented 4 years ago

I know nflfastR still doesn't have position designations native in their package due to new IDs. This could help out their work big time too.

tanho63 commented 4 years ago

That's an interesting idea. Possibly more helpful in historical terms than it is for going forward

trojanguard25 commented 3 years ago

I think I've mapped the vast majority of gsis_ids and pbp_ids in nflfastR 2020 play-by-play data to mfl_ids. I'd like to expand missing_ids.csv to have columns for gsis_ids and pbp_ids. Does that seem like the right thing to do so those ids can get populated into dbplayerids.csv?

tanho63 commented 3 years ago

I think I've mapped the vast majority of gsis_ids and pbp_ids in nflfastR 2020 play-by-play data to mfl_ids. I'd like to expand missing_ids.csv to have columns for gsis_ids and pbp_ids. Does that seem like the right thing to do so those ids can get populated into dbplayerids.csv?

@trojanguard25 that would be helpful! If you have a PR for these I can take a look at the integration