BurntSushi / nfldb

A library to manage and update NFL data in a relational database.
The Unlicense
1.08k stars 264 forks source link

All Jaguar players are team 'UKN' #154

Open ntulip opened 8 years ago

ntulip commented 8 years ago

I noticed that there are no players in the table who play for Jacksonville. All players show team = 'UKN'

ochawkeye commented 8 years ago

Well isn't that interesting...

import nfldb
db = nfldb.connect()
q = nfldb.Query(db)
q.game(season_year=2015, season_type='Regular', team='JAC', week=1)

for play in q.as_plays():
    print play.pos_team, play
    for pp in play.play_players:
        print '\t', pp.team, pp.player, pp
JAC (JAC, OWN 20, Q4, 1 and 10) (2:41) (Shotgun) B.Bortles pass short middle intended for C.Harbor INTERCEPTED by T.Davis (M.Addison) at JAC 20. T.Davis to JAC 19 for 1 yard (T.Yeldon).
        JAC Blake Bortles (UNK, UNK) {'passing_int': 1, 'passing_att': 1, 'passing_incmp': 1}
        CAR Thomas Davis (CAR, OLB) {'defense_pass_def': 1, 'defense_int': 1, 'defense_int_yds': 1}
        JAC T.J. Yeldon (UNK, UNK) {'defense_tkl': 1}
        JAC Clay Harbor (NE, TE) {'receiving_tar': 1}
        CAR Mario Addison (CAR, DE) {'defense_pass_def': 1}

Harbor just signed with New England on March 30th, so this could have been pretty recently broken. Did NFL.com recently start calling them "JAX" instead of "JAC"?

ochawkeye commented 8 years ago

All players for the Rams are in this same state since nfl.com now redirects http://www.nfl.com/teams/roster?team=STL to http://www.nfl.com/teams/roster?team=LA

ntulip commented 8 years ago

The Jaguars changed in 2014/2015 to JAX from JAC. I should of mentioned that the query I ran was against the players table and not the stats. I noticed that I could update (maybe not reliable) the players team with the data from plays but wanted to open this issue and find a fix for the actual import/sync.

hpadmanabhan commented 8 years ago

Posted a resolution for this under BurntSushi/nflgame#201 for anyone interested.

Until this gets officially patched the above works as a simple workaround locally.

ntulip commented 8 years ago

Thanks!