-
Notifications
You must be signed in to change notification settings - Fork 21
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
"Conference only" filter in KenPom Scouting Reports #63
Comments
Those are tricky, as it requires interacting with the page in a difficult
way. We'd like to add them, but it's not straight forward. I can't remember
if we figured out a way to do so on the other pages yet or not.
We'll probably push a release before the season kicks off fully to adjust
for changes he's made to the site since the end of last season.
…On Mon, Oct 23, 2023, 3:23 PM ddogwoofwoof ***@***.***> wrote:
In KenPom's scouting reports, there is a "Conference only" filter.
Unfortunately, I don't see this filter option using the summary.py file. Do
you know if this is possible to include? I inspected KenPom's website, but
struggled to find the elements that include this info.
Thank you!
Derek
—
Reply to this email directly, view it on GitHub
<#63>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACOAQNAZ746EWFAYCQH2GXTYA3G2TAVCNFSM6AAAAAA6MUMIZGVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2TOOJWGI3TONI>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
That makes sense. I'll take a stab at it as well and create a pull request if I can make progress. Thank you! |
I believe what you're referring to is something that has already had work put forward in #40. It's currently stalled for a variety of reasons, the biggest of which being that I'm not convinced we should take additional overhead to execute arbitrary JavaScript for support of such a small sliver of the site. I have a local branch where I've tried to do this purely with the current dependency set and parsing some of the JS content with a thoughtful RegExp but it's not yet all the way there. If you are so inclined, please feel free to try to pull together a PR and we'd be more than happy to consider it. |
I just now remembered my attempt at this has actually been pushed to the enhancement/team-scouting-report branch. I don't recall exactly how much more work is required to get this ready for release (it may be as little as some test coverage?), but the fundamental underpinnings are there if you want to build off that. If not, I will try my best to get back around to this soon. |
@esqew this is perfect!! Thank you so much for sharing! |
In KenPom's scouting reports, there is a "Conference only" filter. Unfortunately, I don't see this filter option using the summary.py file. Do you know if this is possible to include? I inspected KenPom's website, but struggled to find the elements that include this info.
Thank you!
Derek
The text was updated successfully, but these errors were encountered: