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

Doesn't seem to use hyperion's config #118

Closed
pmishev opened this issue Sep 18, 2018 · 4 comments
Closed

Doesn't seem to use hyperion's config #118

pmishev opened this issue Sep 18, 2018 · 4 comments

Comments

@pmishev
Copy link

pmishev commented Sep 18, 2018

When playing a video in the Plex app, it doesn't seem to handle the black bars on top and bottom, which is configured in hyperion.json.
Additionally in the configuration I have to setup the number of horizontal and vertical pixels - why do I have to do that, if these are configured in the hyperion config as well?

Perhaps I don't understand how the whole things works - I thought the app just captures the screen and sends the data to the hyperion server for processing and lighting the LEDs?

@Joeboyc2
Copy link

Thats a good point actually and I did wonder why I was being asked when setting up the latest version recently

Be good to understand this please

@abrenoch
Copy link
Owner

Fair question @pmishev, I'll explain.

The app will use that information to determine what quality of image will be sent to hyperion. To maintain the correct aspect ratio, we can only scale by the common divisors for both the width & height of your device's screen. Rather than leaving it up to the user to determine the optimal quality, the app will calculate the lowest quality acceptable for the number of LEDs in your setup. Sending a higher image quality than needed will simply consume more resources on your device without really getting any benefit out of it, so this will do that work for you!

For the black bars, could you make sure the black border threshold in your hyperion settings in greater than 0 (I use 0.01). I'm honestly not sure why that makes the difference but it appears to!

@ninovanhooff
Copy link
Collaborator

ninovanhooff commented Sep 19, 2018 via email

@abrenoch
Copy link
Owner

Haven't heard any updates here, so I'm going to close this. Just respond if there is still a problem.

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

No branches or pull requests

4 participants