-
Notifications
You must be signed in to change notification settings - Fork 43
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
[Bug] ESPN league error when generating report #201
Comments
Hi @h-r-x, thanks for the bug report. I just tried generating a report for a private ESPN league using v16.0.1 (the latest published version), and it worked without issue. There was a change to the third-party library I use to pull ESPN data... could you try running Also thanks for your longtime support, I'm glad that the app is still providing value to people in the FF community! |
Thanks for the reply! I did what you suggested and it did update a few missing requirements. Still getting the same exact error as mentioned above unfortunately though. I also tried a fresh clone, following all the steps again from the beginning, and that returned the same error again as well. |
@h-r-x did your league start in week 1 or a later week? Also, on line 329 of Your stack trace seems to indicate that the matchup teams are integers, but they should be of type |
Week 1, yes, though this particular league does have an odd number of teams (7) this season, so teams all have 2 Bye weeks. Could that be what is returning integers? Also, I'm having trouble getting an output. When I run the file it had multiple problems, most of which I resolved (simply python version mismatch), but I am now getting:
Another thing I noticed is that the out log states:
I double checked compose.yaml and it does correctly show |
Were you trying to run only that file? My apologies if I was unclear, what I meant was for you to add the print statement and then run the report as you normally would. Also, that's odd about the version... am I correct in assuming you're using docker to run the report? Lastly, I have actually never tested the app on leagues without even numbers of teams... so I honestly have no idea how it will behave, so that seems a likely culprit for the missing team data, since the report has no concept of bye weeks (it does not support being run during fantasy playoffs, for instance). Without the ability to see what the data returned from the ESPN API looks like in your specific scenario, I'll have trouble trying to support it. If you're interested though and don't mind, you can always add me as a comanager to your team on ESPN so I can get at your private league data to test it out, but that's up to you. If you want to try that then let me know and I can share my ESPN FF account email that you would invite. |
Yeah I'm using docker. Also I tried that, but it's not printing it when attempting to generate a report. All I get back is the same error as above. Yeah I'm okay with you looking into it if you'd like to verify if that's the issue. |
Yeah now that I know it's a 7 team league with bye weeks, I'm positive that is the reason you're getting your error. As I mentioned, I have never had the opportunity to test the app on a league with odd numbers of teams and bye weeks, so it's fairly safe to assume it'll break. That being said, if you're interested in me looking into how feasible it would be to support your configuration and you're comfortable with me existing as a "shadow" co-manager on your team in that league in order to access the data, then you can email me at |
Thanks, sent you an email. I'll leave this issue open to you to decide how to close it once you can get in and diagnose it better. Appreciate the support |
Description of Issue
Reports have worked flawlessly on previous versions (used throughout the 2022 season), but after updating to current version for 2023 season, I have been getting the following issue when attempting to generate reports for a private ESPN league:
I attempted to generate for default week as well as numerous other weeks (1-8) to see if that would help, but no luck.
Fantasy Football Platform
ESPN
Operating System
Windows
Also just wanted to say that I've loved this repo and thank you for your work on this.
The text was updated successfully, but these errors were encountered: