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

Blurhashes #1482

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Blurhashes #1482

wants to merge 5 commits into from

Conversation

sevenrats
Copy link
Member

@sevenrats sevenrats commented Nov 10, 2023

enabling this new feature will raise the floor of loading time a bit, but is still probably an improvement for a large number of devices. the negative impact on loading times can be mitigated in the long term by making the number of items fetched at once configurable, and users can just turn it down when using blurhashes if they wish.

@sevenrats sevenrats requested a review from a team as a code owner November 10, 2023 05:55
@cewert cewert added the new-feature A new feature that currently doesn't exist. label Nov 10, 2023
@jellyfin-bot jellyfin-bot added the merge-conflict This PR has a merge conflict label Nov 11, 2023
@jellyfin-bot
Copy link
Contributor

This pull request has merge conflicts. Please resolve the conflicts so the PR can be reviewed. Thanks!

@jellyfin-bot jellyfin-bot removed the merge-conflict This PR has a merge conflict label Nov 11, 2023
@anthonylavado
Copy link
Member

For what it's worth, I've just tested this on the "company" (read: Jellyfin) Roku test device - a Streaming Stick+ (3810CA, released Sept 2019) purchased in Feb 2020.

I performed a normal test with the setting disabled and enabled against my test Jellyfin library, restarting the Roku device in-between setting changes, to ensure that it was starting from a cold boot. With the setting on, I did not notice any slowdowns across my local network. If there was any paging, it happened at the same points in the longer library lists, and did not appear to take any longer.

If anything, it was actually nice to have the correct colours of the posters that would be loaded, instead of the template blue that stuck out when scrolling through quickly.

I like this change, and would recommend it.

@jellyfin-bot
Copy link
Contributor

This pull request has been inactive for 21 days and will be automatically closed in 7 days if there is no further activity.

@jellyfin-bot jellyfin-bot added the stale This issue/PR has gone stale. label Dec 8, 2023
@sevenrats
Copy link
Member Author

shoot bot

@jellyfin-bot jellyfin-bot removed the stale This issue/PR has gone stale. label Dec 9, 2023
@cewert cewert changed the base branch from unstable to master December 14, 2023 15:06
@jellyfin-bot
Copy link
Contributor

This pull request has been inactive for 21 days and will be automatically closed in 7 days if there is no further activity.

@jellyfin-bot jellyfin-bot added the stale This issue/PR has gone stale. label Jan 7, 2024
@jellyfin-bot
Copy link
Contributor

This pull request has been closed because it has been inactive for 28 days. You may submit a new pull request if desired.

@joshuaboniface
Copy link
Member

@cewert @1hitsong Any additional feedback on this one?

@jellyfin-bot jellyfin-bot removed the stale This issue/PR has gone stale. label Jan 17, 2024
@1hitsong
Copy link
Member

This messes up the View All posters for TV Shows in genre view.

This PR
image

Expected
image

@1hitsong
Copy link
Member

1hitsong commented Jan 28, 2024

I see the console print for music artists, but they don't render. It goes straight from placeholder to final rendered image.

Same with music genres.

@1hitsong 1hitsong closed this Jan 28, 2024
@1hitsong 1hitsong reopened this Jan 28, 2024
@jellyfin-bot
Copy link
Contributor

This pull request has been inactive for 21 days and will be automatically closed in 7 days if there is no further activity.

@jellyfin-bot jellyfin-bot added stale This issue/PR has gone stale. merge-conflict This PR has a merge conflict labels Feb 19, 2024
@jellyfin-bot
Copy link
Contributor

This pull request has merge conflicts. Please resolve the conflicts so the PR can be reviewed. Thanks!

@jellyfin-bot jellyfin-bot removed the stale This issue/PR has gone stale. label Feb 21, 2024
@sevenrats
Copy link
Member Author

not stale. working on it still.

@jellyfin-bot
Copy link
Contributor

This pull request has been inactive for 21 days and will be automatically closed in 7 days if there is no further activity.

@jellyfin-bot jellyfin-bot added the stale This issue/PR has gone stale. label Mar 19, 2024
@sevenrats
Copy link
Member Author

not stale

@jellyfin-bot jellyfin-bot removed the stale This issue/PR has gone stale. label Mar 20, 2024
@jellyfin-bot
Copy link
Contributor

This pull request has been inactive for 21 days and will be automatically closed in 7 days if there is no further activity.

@jellyfin-bot jellyfin-bot added the stale This issue/PR has gone stale. label Apr 10, 2024
@sevenrats
Copy link
Member Author

oh 21 days already. when I finally finish this PR I will need to find a new way to keep track of the weeks.

@cewert
Copy link
Member

cewert commented Apr 12, 2024

I'm converting this to a draft PR so the bot will stop marking this as stale. Please convert it back whenever it's ready for review again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-conflict This PR has a merge conflict new-feature A new feature that currently doesn't exist.
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

6 participants