Code Monkey home page Code Monkey logo

Comments (5)

bratanon avatar bratanon commented on June 12, 2024

This also seems related home-assistant/frontend#16666

from custom-ui.

Mariusthvdb avatar Mariusthvdb commented on June 12, 2024

doesnt the 16666 prove it has nothing to do with custom-ui? and rather its a more generic HA issue?
unless the user is also using custom-ui, it would look for solving that in HA

doesnt mean I am not hoping custom-ui could be optimized, and I have indicated before there's a constant updating going I would love to prevent

as you say:

So every time some state change, we are there and looping thru all states and checking if the entity the state belongs to have template in the list of attributes.

if this PR you suggest would actually stop that from happening, and only check the required states, that would be awesome.

from custom-ui.

bratanon avatar bratanon commented on June 12, 2024

doesnt the 16666 prove it has nothing to do with custom-ui? and rather its a more generic HA issue?
unless the user is also using custom-ui, it would look for solving that in HA

Yes and no. As custom-ui is constantly "updating" all its customization's object because of the broken if statements, that is making the HA bug even worse. Custom-ui is constantly triggering the bug by its unnecessary updates.

doesnt mean I am not hoping custom-ui could be optimized, and I have indicated before there's a constant updating going I would love to prevent

This doesn't solve all of them, but at least one of them.

if this PR you suggest would actually stop that from happening, and only check the required states, that would be awesome.

No it doesn't, this PR fixes the BROKEN if statement that will prevent the updates later down the stack if there is checks in place. To me it seems like you don't understand the code in PR at all, is that why you are unsure about this?

At least it will prevent custom-ui to change states that it doesn't need to be changed. Remember, we are high-jacking the method so we don't "own" anything here.

from custom-ui.

Mariusthvdb avatar Mariusthvdb commented on June 12, 2024

its not that I dont understand, its more the fact I want to make sure we are not tailor-made 'fixing' issues in other software that should be generically fixed in that other software.

I can see the value ofc of preventing those useless updates. as a matter of fact, I believe the complete plugin should be reviewed on that aspect, and adjusted.

tbh, Ive stopped using it altogether, moved my icon changes to card-mod, and have created a tiny separate custom-icon-color plugin to do only that (it still uses the old code, attaching the attribute and changing the state badge with it)

having said all of that, I'll approve, and would like to ask if you can write a short and succinct description please, without referring to the Map issue in HA.

thanks for you in depth explanation above, and effort to keep this plugin up to date.

hope you will be able to have a look at the other open issue, where the templates, or more specifically the templates results no longer show as a result when templated in Jinja.
Ill tag you there, maybe you can have a look

from custom-ui.

bratanon avatar bratanon commented on June 12, 2024

....would like to ask if you can write a short and succinct description please

Addressed an issue where state objects were inadvertently updated as a result of a comparison error.

from custom-ui.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.