Generic Level Server Bugs

RE: Generic Level Server Bugs

david_owtschinnikow — Thu, 07 Feb 2019 13:21:29 GMT

OK, thank you very much for your support

RE: Generic Level Server Bugs

Hung Bui — Thu, 07 Feb 2019 12:56:17 GMT

Hi,

1) Yes you can do what you have done.

2) After a double check we confirmed that it's a bug. You can use TRANSITION_TIME_MAX_MS

3) I think that's the way it should be for the Move Set command. The present level should by the end of transition time stop at the INT16_MAX or INT16_MIN. Or keep wrapping over like what we have implemented. My understanding is that there is no target level that the state need to get to for this command. If you want a target level you should use the Level Set.

RE: Generic Level Server Bugs

david_owtschinnikow — Wed, 06 Feb 2019 09:03:24 GMT

Thank you for your response

1) I understand, but in my case (I want to stop the movement when the transition time is completed) I don’t find another way of stopping the movement other than the way I posted, because inside “g_transition_complete()”, if the condition to stop is NOT( TRANSITION_MOVE_SET), then a move set never stops. Besides, the target level is already INT16_MAX or INT16_MIN.

2) In that case, I cannot handle transition times higher than TRANSITION_TIME_STEP_100MS_MAX, which is 6200ms (not encoded). The comparison is done between decoded values, in ms, not between encoded values. According to the spec., the maximum transition time is 10.5 hours. I don’t see why the maximum should be 6200ms?

3)I have undone my change according to what you said, but if I do so, then no matter what parameters I send to the server (transition time and delta level), the “present level” at the end of the transition is always equal to the “target level”, which is INT16_MAX or INT16_MIN; I don’t think this is the correct behaviour. What would then be the correct solution?

Thank you very much

RE: Generic Level Server Bugs

Hung Bui — Tue, 05 Feb 2019 15:04:39 GMT

Hi again David,

Regarding your questions:

1. According to spec, Mesh model v1.0:

When the Generic Level state is not bound to another state, the overflow/underflow handling is
implementation-specific. Some Generic Level Servers may stop at their maximum or minimum levels,
and some may wrap around.

Since this is implementation specific, we choose to do continuous movement showing the wrap around behavior. You can modify this behavior to stop at INT16_MAX or INT16_MIN values.

2. TRANSITION_TIME_STEP_100MS_MAX is the correct value in milliseconds and not the encoded value. You should not use the TRANSITION_TIME_MAX_MS. The reason is the transition number of steps is limited to 0x00 to 0x3E according to spec.

3. Target level should be 32767 for positive move values (level increases continuously) and should be -32768 for negative move values. This is according to the spec:

Upon receiving a Generic Move Set message, the Generic Level Server shall respond with a Generic Level Status message (see Section 3.3.2.2.5). The target Generic Level state is the upper limit of the Generic Level state when the transition speed is positive, or the lower limit of the Generic Level state
when the transition speed is negative.

4. It's a bug and we have reported it. Fix is coming on v3.2

RE: Generic Level Server Bugs

Hung Bui — Thu, 31 Jan 2019 10:12:02 GMT

Hi David,

I'm sorry for the late response. I'm forwarding your suggested fixes to our team and I will get back to you ASAP.