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

Enable NFL Team to be directly accessible #406

Merged
merged 1 commit into from
Apr 26, 2020
Merged

Conversation

roclark
Copy link
Owner

@roclark roclark commented Apr 25, 2020

Instead of requiring users to go through the Teams class to get a specific team, the NFL modules now enable a specific team to be directly queried by using the Team class. This reduces computational complexity by removing the need to instantiate every team while also making it more intuitive for users.

Related to #360

Signed-Off-By: Robert Clark robdclark@outlook.com

@roclark roclark added the enhancement New feature or request label Apr 25, 2020
@roclark roclark added this to the Release 0.6.0 milestone Apr 25, 2020
@roclark roclark self-assigned this Apr 25, 2020
@roclark roclark force-pushed the direct-nfl-team-module branch 2 times, most recently from 79f7ded to ae95c47 Compare April 25, 2020 23:48
@codecov
Copy link

codecov bot commented Apr 25, 2020

Codecov Report

Merging #406 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #406   +/-   ##
=======================================
  Coverage   99.97%   99.97%           
=======================================
  Files          52       53    +1     
  Lines       10683    10699   +16     
=======================================
+ Hits        10680    10696   +16     
  Misses          3        3           
Impacted Files Coverage Δ
sportsreference/nfl/nfl_utils.py 100.00% <100.00%> (ø)
sportsreference/nfl/teams.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0ba4b6f...959ab23. Read the comment docs.

Instead of requiring users to go through the Teams class to get a
specific team, the NFL modules now enable a specific team to be directly
queried by using the Team class. This reduces computational complexity
by removing the need to instantiate every team while also making it more
intuitive for users.

Signed-Off-By: Robert Clark <robdclark@outlook.com>
@roclark roclark merged commit a48e3ea into master Apr 26, 2020
@roclark roclark deleted the direct-nfl-team-module branch April 26, 2020 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant