Light will turn off but not on about smartthings-mqtt-bridge

[ikonron]

I could only get mine to turn back on by changing the server.js file in /usr/local/lib/node_modules/smartthings-mqtt-bridge

I remmed out 260 -267 and I was able to turn it back on with HA:

This is why it wasn't turning on.
info: Incoming message from MQTT: smartthings/Basement Light 1/switch = off
info: Incoming message from MQTT: smartthings/Basement Light 1/switch = on
info: Passing level instead of switch on

Must have something to do with the brightness level st is reporting to the bridge making it think it is already on:.

Now on to figure out why the brightness level reporting or control isn't working.

[stjohnjohnson]

Interesting, because SmartThings should accept the brightness and turn the light on.
https://github.com/stjohnjohnson/smartthings-mqtt-bridge/blob/master/server.js#L260-L267

http://docs.smartthings.com/en/latest/capabilities-reference.html#switch-level
See comment:

    myswitch.setLevel(90) // also turns on the switch

[mehuman]

Taking a second look at this, I'm now getting this error in the live log. It happens when setting either the switch or trying to set the level.
Tried restarting the bridge and am getting the same error.

10:21:48 AM: error groovy.lang.MissingMethodException: No signature of method: java.lang.String.div() is applicable for argument types: (java.lang.Integer) values: [100]
Possible solutions: is(java.lang.Object), drop(int), wait(), trim(), find(), size() @ line 85

[stjohnjohnson]

What devicetype do you have for your light? It sounds like theres a bug in that code.

[ikonron]

[image: Inline image 1]

Osram LIGHTIFY® LED A19 Tunable White
I also tried Zigbee Dimmer as well.
Both of my lights are the same Osram. Experiencing the same results on
both. One thing I did notice was that when I tried dimming, the slider
moved in the app but the lamp did nothing. I haven't had time to mess with
it but hope to this weekend.

On Fri, May 20, 2016 at 1:33 PM, St. John Johnson [email protected]
wrote:

What devicetype do you have for your light? It sounds like theres a bug in
that code.

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#40 (comment)

[scottosx]

I can verify that this condition is occurring as well. Commenting out lines 260 -267 restores the ability to turn the lights on and off. However, I still cannot set the level using the /level topic.

I do get the status update from the console (if running without PM2)
info: Incoming message from MQTT: smartthings/Bedroom Light 1/level = 50

I do not see any change in the level slider in the app as indicated by Ikonron.

I have tested this on Osram Lightify as well as LIFX bulbs.

Thanks for any insight and this awesome addition to the Smartthings arsenal.

[scottosx]

In the log I do see a similar error as reported above:

10:36:55 AM: debug Received device event from bridge: [name:Bedroom Light 1, value:50, type:level]
10:36:55 AM: error groovy.lang.MissingMethodException: No signature of method: java.lang.String.div() is applicable for argument types: (java.lang.Integer) values: [100]
Possible solutions: is(java.lang.Object), drop(int), wait(), trim(), dump(), size() @ line 527

[mehuman]

The lights I'm using are the Cree Connected Smartbulbs.
http://creebulb.com/connected-60-watt-replacement-soft-white

[stjohnjohnson]

I'm thinking the problem is something in the Device Type for your light bulb. We're sending an integer, while it looks like the device requires a string. Very strange. Not sure how to proceed here.

[kevinoconnor7]

I ran into this as well. It seems that the app is passing the bulb a string when it expects an int. I fixed it by updating the levelAction to call setLevel(value as int).

That resolved the issue for me, but I'm not sure it's going to be correct for all device types.

[mehuman]

^ This fix worked for me as well. I updated mine to read:

def actionLevel(device, attribute, value) {
device.setLevel(value)
device.setLevel(value as int)
}

Everything appears to be working, I still get an error about the value not being an int but the light turns on and I can adjust the level with the slider in HA.

Logs turning light on and adjusting the level:

0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:24 AM: debug [raw:0104 0008 0A 01 0140 00 4917 00 00 0000 0B 01 0400, profileId:0104, clusterId:0008, clusterInt:8, sourceEndpoint:0A, destinationEndpoint:01, options:0140, messageType:00, dni:4917, isClusterSpecific:false, isManufacturerSpecific:false, manufacturerId:0000, command:0B, direction:01, data:[04, 00]]
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:24 AM: debug DID NOT PARSE MESSAGE for description : catchall: 0104 0008 0A 01 0140 00 4917 00 00 0000 0B 01 0400
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:24 AM: debug level updated successfully
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:24 AM: debug description is catchall: 0104 0008 0A 01 0140 00 4917 00 00 0000 0B 01 0400
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:24 AM: debug description is read attr - raw: 49170A00080A00000020BC, dni: 4917, endpoint: 0A, cluster: 0008, size: 0A, attrId: 0000, result: success, encoding: 20, value: bc
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:23 AM: debug description is catchall: 0104 0006 0A 01 0140 00 4917 00 00 0000 0B 01 0100
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:23 AM: error groovy.lang.MissingMethodException: No signature of method: java.lang.String.div() is applicable for argument types: (java.lang.Integer) values: [100]
Possible solutions: drop(int), is(java.lang.Object), wait(), trim(), dump(), find()
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:10 AM: debug description is catchall: 0104 0006 0A 01 0140 00 4917 00 00 0000 0B 01 0100
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:08 AM: debug [raw:0104 0008 0A 01 0140 00 4917 00 00 0000 0B 01 0400, profileId:0104, clusterId:0008, clusterInt:8, sourceEndpoint:0A, destinationEndpoint:01, options:0140, messageType:00, dni:4917, isClusterSpecific:false, isManufacturerSpecific:false, manufacturerId:0000, command:0B, direction:01, data:[04, 00]]
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:08 AM: debug DID NOT PARSE MESSAGE for description : catchall: 0104 0008 0A 01 0140 00 4917 00 00 0000 0B 01 0400
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:08 AM: debug level updated successfully
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:08 AM: debug description is catchall: 0104 0008 0A 01 0140 00 4917 00 00 0000 0B 01 0400
0a40c8f8-5f19-4376-8607-ddc933bcc0b9 10:24:08 AM: error groovy.lang.MissingMethodException: No signature of method: java.lang.String.div() is applicable for argument types: (java.lang.Integer) values: [100]
Possible solutions: is(java.lang.Object), drop(int), wait(), trim(), dump(), size()

[kevinoconnor7]

Just make the code:

def actionLevel(device, attribute, value) {
  device.setLevel(value as int)
}

That should work and you shouldn't get any errors

[mehuman]

You said earlier "That resolved the issue for me, but I'm not sure it's going to be correct for all device types." I was thinking that even though it errors it should make it universal for all device types to have both calls in. Thoughts?

[kevinoconnor7]

It's possible that an integer is always expected there, I just don't have enough devices to test with. If it works in your environment only passing in an int then I'd just leave it.

[stjohnjohnson]

Agreed, looks like my bad. My devicetypes were fixing the string->int, but not Cree. I have a PR open and @jer should be able to review it today.

Good find.

[stjohnjohnson]

• [email protected]