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

Language filter appears to not work at all #724

Closed
2 tasks done
chrisheib opened this issue May 21, 2020 · 5 comments
Closed
2 tasks done

Language filter appears to not work at all #724

chrisheib opened this issue May 21, 2020 · 5 comments

Comments

@chrisheib
Copy link

Bug report

  • I understand the difference between Streamlink and Streamlink Twitch GUI.
  • This is a bug report and I have read the contribution guidelines.

Environment details

Operating system and version:
Windows 10

Streamlink Twitch GUI version:
1.9.1 (Set to German)

Streamlink version:
1.3.1

Description

The language filter is not being applied to streams. (Language fading is working correctly.)
Streams with languages that are not selected are being shown.

Expected / Actual behavior

Streams with languages that are not selected don't get shown in the streamlist.

Reproduction steps

  1. Go to Options.
  2. Set Language Filter to 'Filter'
  3. Select 'German' and 'English' only.
  4. Apply Settings
  5. Go to Streams
  6. Spanish and Korean streams appear in the list.

Additional comments, screenshots, etc.

grafik

grafik

@bastimeyer
Copy link
Member

@chrisheib
Copy link
Author

Can't the filtering be clientside theoretically tho?

@bastimeyer
Copy link
Member

No.

If you search for "client side filtering" on the issue tracker, you will find plenty of comments:
#564, #421

@chrisheib
Copy link
Author

Thanks alot for the replies, I understand now. Guess some multithreaded querying with multiple languages in parrallel wouldn't work either, I don't know enough about the API limitations.

Do you have plans for a new release, or should I rather build the current Version myself for now?

Also thank you very much for maintaining this project, it's a really nice tool <3

@bastimeyer
Copy link
Member

multithreaded querying with multiple languages in parrallel

Those massive API requests are not possible since Twitch is already rate limiting the new API (which btw is a total disaster) and has recently tried to also introduce it to the old one which is currently used here. They've backpedaled though after receiving a shitstorm on their forums. Still, filtering features and the likes can only be implemented here if they are supported on their API and doing multiple network requests to achieve just one goal is always bad. See my comment here regarding a recent temporary change on the followed streams API endpoint:
https://discuss.dev.twitch.tv/t/kraken-streams-followed-not-ordered-by-viewers/23772/8

Do you have plans for a new release

Probably this weekend. I need to check a couple more things first, and I don't have the time today.

or should I rather build the current Version myself for now

You can always build from the master branch. The more people who are running the latest dev version, the better the chances for finding bugs/issues which I've missed.

it's a really nice tool

thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants