SetCurves

[robert-b]

Recently there has been a fix applied for SetCurves: #291
Unfortunetly it still does not work.
The described json document is wrong and fixing this has not solved the issue.

[geduxas]

it's not in released firmware yet. so you should need to wait, or test alpha firmware..

[robert-b]

then it is time to test - as i need this functionality.
thanks for your help.

[IgorYbema]

Sorry but the fix isn't working good enough. It breakes other rules so for now it isn't ready for the next firmware release.

[robert-b]

surprise

[stumbaumr]

As far as I understand modifying the heat curves already works since release 3.8.
Sample from my rules:
@SetCurves = concat('{zone1:{heat:{target:{high:', $targetAfterDHW ,',low:', $targetAfterDHW ,'}}}}');

Or am I missing something here?

[robert-b]

i have no idea - but i can test the snippet.
up to now - nothing has been accepted when saving my rules.

[stumbaumr]

You can test my rules... the snippet alone will not work...

[robert-b]

ok - this snippet is bad:
@SetCurves = concat('{"zone1":{"heat":{"target":"high":', @Z1_Heat_Curve_Target_High_Temp, ',"low":', @Z1_Heat_Curve_Target_Low_Temp, '}}}}');

but this:
@SetCurves = concat('{zone1:{heat:{target:{high:', $target ,',low:', $low ,'}}}}');
works
already tested.
my updated script works.
very interesting and thanks a lot.

[stumbaumr]

So you should be able to close the issue then, right?

[McMagellan]

I can confirm that SetCurves used in Rules has worked very well without any problems since 3.8.
For example: @SetCurves = concat('{zone1:{heat:{target:{high:',#VLRef,',low:',#VLRef,'}}}}');
Thanks for the implementation.

Are there any considerations about setting up a "Rules Wiki" where you can look up small code examples?

[robert-b]

i can't close the issue because there is a difference between the actual implementation and the documentation.
both have to be in sync.