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

nflgame.teams cleanup; too many commas ValueError; reduce() TypeError #328

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

ochawkeye
Copy link
Contributor

  • Reordering SD above LAC avoids overwriting Charger players with team of SD.
  • Made New York and Los Angeles team rows a bit more consistent; Full official team name is not always t[3].
  • Removed the teams that arbitrarily included abbreviations with periods in them, opting instead to strip the periods when running nflgame.standard_team(TEAM).
  • Avoid crashing with TypeError when collecting players over an empty list of games.
  • Sneaking in Charles Leno, Jr. fix once and for all.

@snake-plissken
Copy link
Contributor

Many Bothans died for that Charles Leno, Jr fix.

@mickeelm
Copy link

@ochawkeye Hey, this code seems like it really should be merged. Since this PR won't be accepted here, would you mind if it got merged into derek-adair's fork instead? Either if you submit a PR or if I did.

derek-adair pushed a commit to derek-adair/nflgame that referenced this pull request Aug 23, 2018
* 22nd Annual Charles Leno Junior Memorial Award(TM)

* Cleanup nfl team list in __init__.py

Removed arbitary teams that included a dot between letters as alternate name.
Reordered SD and LAC teams so non-existant SD team quit overwriting LAC.

* Fix TypeError when reduce() is performed on an empty list

* _create_schedule demonstration
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

Successfully merging this pull request may close these issues.

3 participants