Got it, I'll fix that. I should have noted that I changed this in my pull comments to make the change to 0.0 or 1.0 only, more clear. You're ok with the more direct comparison of 0.0 and 1.0 (using the correct code)? Old code was ok with 0.0->0.5 for off and 0.5->1.0 for on. I was concerned that given mavlink spec says only 0.0 and 1.0 are valid, anything other values should really error out since who knows that they intended.