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

Bug with segments color rule #34

Open
lucasth24 opened this issue Apr 29, 2021 · 4 comments
Open

Bug with segments color rule #34

lucasth24 opened this issue Apr 29, 2021 · 4 comments

Comments

@lucasth24
Copy link

Describe the bug
When trying to set the Min and Max values using the msg.control.seg1/2, i've notice a bug with the color rule. It doesnt respect the limits imposed by the msg.control.seg1/2. After testing, looks like there's some problem related to odd/even numbers, cuz they respond different.

To Reproduce
Just enter some msg.control.seg like: 166175 and 167176 and move 1 before the lower limit (168) and 1 above the higher limit (177).
Expected behavior
For exemple, when setted to 166~175 it should be red from 0 to 165, green from 166 to 175, and red from 176 to 250.

Screenshots
image

Great work by the way!!

Environment and versions

OS: windows 10
Microsoft Edge
Node-RED version: 1.1.3
Dashboard version: 2.24.2
ui-level version: 0.1.40

Additional context
Add any other context about the problem here.

@hotNipi
Copy link
Owner

hotNipi commented Apr 30, 2021

Would you please create a little test flow for me.

@lucasth24
Copy link
Author

Teste code bellow:

[{"id":"c951ec63.546ad","type":"ui_level","z":"e4085288.0f8a3","group":"b3261d37.ed8d1","order":0,"width":"3","height":"8","name":"1","label":"1","colorHi":"#e60000","colorWarn":"#00ff00","colorNormal":"#e60000","colorOff":"#595959","min":"50","max":"250","segWarn":"","segHigh":"","unit":"","layout":"sv","channelA":"","channelB":"","decimals":0,"animations":"soft","shape":2,"colorschema":"valuedriven","textoptions":"custom","colorText":"#eeeeee","fontLabel":"2","fontValue":"","fontSmall":"","colorFromTheme":true,"textAnimations":false,"hideValue":false,"tickmode":"segments","peakmode":false,"property":"payload","peaktime":3000,"x":290,"y":140,"wires":[]},{"id":"d7d12077.fa5a2","type":"inject","z":"e4085288.0f8a3","name":"seg1","props":[{"p":"control.seg1","v":"166","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":100,"y":80,"wires":[["c951ec63.546ad"]]},{"id":"bd2b9aa5.d94f68","type":"inject","z":"e4085288.0f8a3","name":"seg2","props":[{"p":"control.seg2","v":"175","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":100,"y":140,"wires":[["c951ec63.546ad"]]},{"id":"1bef74e3.10074b","type":"inject","z":"e4085288.0f8a3","name":"","props":[{"p":"payload"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","payload":"176","payloadType":"num","x":100,"y":200,"wires":[["c951ec63.546ad"]]},{"id":"6a3065d6.3c603c","type":"ui_level","z":"e4085288.0f8a3","group":"b3261d37.ed8d1","order":0,"width":"3","height":"8","name":"2","label":"2","colorHi":"#e60000","colorWarn":"#00ff00","colorNormal":"#e60000","colorOff":"#595959","min":"50","max":"250","segWarn":"","segHigh":"","unit":"","layout":"sv","channelA":"","channelB":"","decimals":0,"animations":"soft","shape":2,"colorschema":"valuedriven","textoptions":"custom","colorText":"#eeeeee","fontLabel":"2","fontValue":"","fontSmall":"","colorFromTheme":true,"textAnimations":false,"hideValue":false,"tickmode":"segments","peakmode":false,"property":"payload","peaktime":3000,"x":290,"y":340,"wires":[]},{"id":"757789ab.2c9118","type":"inject","z":"e4085288.0f8a3","name":"seg1","props":[{"p":"control.seg1","v":"166","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":100,"y":280,"wires":[["6a3065d6.3c603c"]]},{"id":"5394ee9a.5285c","type":"inject","z":"e4085288.0f8a3","name":"seg2","props":[{"p":"control.seg2","v":"175","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":100,"y":340,"wires":[["6a3065d6.3c603c"]]},{"id":"a90c7520.34f618","type":"inject","z":"e4085288.0f8a3","name":"","props":[{"p":"payload"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","payload":"166","payloadType":"num","x":100,"y":400,"wires":[["6a3065d6.3c603c"]]},{"id":"8e13dbcd.bed438","type":"ui_level","z":"e4085288.0f8a3","group":"b3261d37.ed8d1","order":0,"width":"3","height":"8","name":"3","label":"3","colorHi":"#e60000","colorWarn":"#00ff00","colorNormal":"#e60000","colorOff":"#595959","min":"50","max":"250","segWarn":"","segHigh":"","unit":"","layout":"sv","channelA":"","channelB":"","decimals":0,"animations":"soft","shape":2,"colorschema":"valuedriven","textoptions":"custom","colorText":"#eeeeee","fontLabel":"2","fontValue":"","fontSmall":"","colorFromTheme":true,"textAnimations":false,"hideValue":false,"tickmode":"segments","peakmode":false,"property":"payload","peaktime":3000,"x":770,"y":140,"wires":[]},{"id":"96e1b14c.b2ea5","type":"inject","z":"e4085288.0f8a3","name":"seg1","props":[{"p":"control.seg1","v":"167","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":580,"y":80,"wires":[["8e13dbcd.bed438"]]},{"id":"d4b67598.b37008","type":"inject","z":"e4085288.0f8a3","name":"seg2","props":[{"p":"control.seg2","v":"176","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":580,"y":140,"wires":[["8e13dbcd.bed438"]]},{"id":"1e1b2cb9.7b77b3","type":"inject","z":"e4085288.0f8a3","name":"","props":[{"p":"payload"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","payload":"168","payloadType":"num","x":580,"y":200,"wires":[["8e13dbcd.bed438"]]},{"id":"891ba684.2d81f8","type":"ui_level","z":"e4085288.0f8a3","group":"b3261d37.ed8d1","order":0,"width":"3","height":"8","name":"4","label":"4","colorHi":"#e60000","colorWarn":"#00ff00","colorNormal":"#e60000","colorOff":"#595959","min":"50","max":"300","segWarn":"","segHigh":"","unit":"","layout":"sv","channelA":"","channelB":"","decimals":0,"animations":"soft","shape":2,"colorschema":"valuedriven","textoptions":"custom","colorText":"#eeeeee","fontLabel":"2","fontValue":"","fontSmall":"","colorFromTheme":true,"textAnimations":false,"hideValue":false,"tickmode":"segments","peakmode":false,"property":"payload","peaktime":3000,"x":770,"y":320,"wires":[]},{"id":"b1a3e70f.c24008","type":"inject","z":"e4085288.0f8a3","name":"seg1","props":[{"p":"control.seg1","v":"167","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":580,"y":260,"wires":[["891ba684.2d81f8"]]},{"id":"b38c6116.b4ed7","type":"inject","z":"e4085288.0f8a3","name":"seg2","props":[{"p":"control.seg2","v":"176","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":580,"y":320,"wires":[["891ba684.2d81f8"]]},{"id":"4d82cea0.0d10a","type":"inject","z":"e4085288.0f8a3","name":"","props":[{"p":"payload"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","payload":"177","payloadType":"num","x":580,"y":380,"wires":[["891ba684.2d81f8"]]},{"id":"b3261d37.ed8d1","type":"ui_group","name":"Group 1","tab":"1465caac.846ec5","order":1,"disp":true,"width":"12","collapse":false},{"id":"1465caac.846ec5","type":"ui_tab","name":"Test","icon":"dashboard","order":4,"disabled":false,"hidden":false}]

@hotNipi hotNipi closed this as completed in b6d17cf May 3, 2021
@hotNipi
Copy link
Owner

hotNipi commented May 3, 2021

Thank you.
Would you please let me know if it works correctly with released version 0.1.41.

@hotNipi hotNipi reopened this May 3, 2021
@lucasth24
Copy link
Author

Hey!
I'm going to stress it and get back to you with some feedbacks!

Thank you very much, amazing job!

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

2 participants