pengtianyue / Solvay_test

0 stars 1 forks source link

Modes - ACTUAL vs TARGET #7

Closed mekcrab closed 8 years ago

mekcrab commented 8 years ago

The confirm for Actions(state_attrs) should be modified to the correct parameter, i.e. MODE.ACTUAL for MODE.TARGET

==>ModeAttribute class should cover this

mekcrab commented 8 years ago

We will need a special AttributeType for modes. Setting a mode writes to MODE.TARGET, checking a mode will refer to MODE.ACTUAL.

What to do about REQ_MODE?

pengtianyue commented 8 years ago

We can just use MODE.TARGET and MODE.ACTUAL. Writing values into REQ_MODE.TARGET just gives a modelock based on the MODE.TARGET, we don't need that.

mekcrab commented 8 years ago

Added this functionality as ModeAttribute in attribute_types branch. Issue closed.