home-assistant / frontend

:lollipop: Frontend for Home Assistant
https://demo.home-assistant.io
Other
4.02k stars 2.75k forks source link

Media Player card - inconsistent coloring for same album art #5202

Closed SeanPM5 closed 4 years ago

SeanPM5 commented 4 years ago

Checklist

The problem

While investigating a different issue, I encountered something else weird...

Had a vertical stack with the media_player.spotify entity and media_player.bedroom_speaker (a Google Home mini) entity, both playing the same song at the same time. Despite the album art being identical, each card got different coloring. And then I checked on my iPhone 11, which also had different coloring than Chrome Mac?

image

I thought at first this might be random, but it doesn't appear to be. Just different colors depending on entity and device or something? idk.

More examples...

image image image image

Expected behavior

Steps to reproduce

  1. Make a vertical stack with media_player.spotify and media_player.cast_device
  2. In Spotify play to the Cast device
  3. Look at the inconsistency

Environment

Problem-relevant configuration

Javascript errors shown in your browser console/inspector

Additional information

zsarnett commented 4 years ago

@balloob

I have noticed this too with the difference in Spotify and Google Home. I think it has to do with the way the images are supplied. Not 100%. The difference between browsers is not something we can really change. It's the way the browser handles images and colors.

But I would have thought the new Generator would have fixed this issue with different colors on the same browser

The reason I think its the image supplied is that if you play that same song 100 times I bet its the same on google home every time and the same on Spotify every time

balloob commented 4 years ago

Which beta is this? We redid the algorithm in 3.

Still, it's being downsampled and then a histogram extracted, this is not deterministic. Combine that with the images being a different format via different sources.

Anyway try again on b3 if you didn't yet.

SeanPM5 commented 4 years ago

This was after updating to b3, all screenshots are from that build. Not sure whether this also occurred on previous betas. Here's the stack I tested it with.

- type: vertical-stack
  cards:

    - type: media-control
      entity: media_player.spotify

    - type: media-control
      entity: media_player.your_google_cast_device

Was just a little strange as the album art looks identical on my screen(s) and the artwork is being supplied from the same source (Spotify). But I don't think it's a big deal, they all look fine to me. Just thought it was worth mentioning.

stale[bot] commented 4 years ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue now has been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.