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

Teams class from the NBA package seems to be broken. #691

Open
Difio3333 opened this issue Jan 19, 2022 · 3 comments · May be fixed by #764
Open

Teams class from the NBA package seems to be broken. #691

Difio3333 opened this issue Jan 19, 2022 · 3 comments · May be fixed by #764

Comments

@Difio3333
Copy link

Describe the bug
When using the following example code from your README.md
`
from sportsipy.nba.teams import Teams

for team in Teams():
print(team.name)
schedule = team.schedule # Request the current team's schedule
for game in schedule:
print(game.date, game.points_scored, game.points_allowed)
`
It always returns:
The requested page returned a valid response, but no data could be found. Has the season begun, and is the data available on www.sports-reference.com?

The other classes seem to work fine.

Desktop (please complete the following information):

  • OS: Mac OS Catalina 10.15.7
  • Sportsipy Version: 0.6.0
@Clews4U
Copy link

Clews4U commented Jan 20, 2022

I'm having the same issue. Teams seems to work for the other sports.

@chriscanal
Copy link

chriscanal commented Jan 24, 2022

Also broken for me, it looks like the dependency PyQuery is not returning anything from the site:

I've spent 6 hours on this and haven't figured out why this isn't working :(

@Difio3333
Copy link
Author

The second answer here fixed it for me in this specific case. But it's no general fix as you'd have to apply it everywhere manually. https://stackoverflow.com/questions/70519889/when-i-run-the-sportsipy-nba-teams-teams-function-i-am-getting-notified-that-the

@vgeek vgeek linked a pull request Dec 1, 2022 that will close this issue
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 a pull request may close this issue.

3 participants