[Tenergy] ProfileRegistrationException: Too Many Profiles about barbecueboss HOT 6 CLOSED

move the profile registration into an initialize() method in the TenergySolaris module. Now, when the controller initializes bluetooth it also calls this method (first) to make sure the profile is registered. This bluetooth setup (creating the controller) only happens once on app start so we shouldn't have any more trouble with this.

from barbecueboss.

I found this note in the BLE documentation related to registerProfile

Registration can fail if too many profiles are registered, the current limit is 3.

That explains why on the third enable we get the error above. This seems to be a limitation in the BLE implementation of connectIQ. If that session registers 3 times it crashes.

I wrapped the registerProfile call in a try-catch. However, this may introduce weird behaviors as the BLE is interpreting as 3 profiles registered for that session.

After you review we can close.

from barbecueboss.

I'm going to move the registration out the constructor for the bluetooth delegate and into a method in the TenergySolis module. It only needs to be done once when the app starts regardless of how many times you connect/disconnect from the device. So really, it should only be call one time ever during the runtime of the app instance.

from barbecueboss.

I see the initialize() call from Controller.mc:123 but it does not find it. It looks like the update to the repo does not automatically update the modules. Can you please push the changes also to the ext/tenergy module?

from barbecueboss.

d'oh, forgot about that. fixed & pushed

from barbecueboss.

Working now. Thanks!

Closing.

from barbecueboss.