search

bersim / OpenKPs

10 stars 9 forks source link

Force use period (.) as decimal separator #5

Closed kumajaya closed 6 years ago

kumajaya commented 6 years ago

For compatibility to some IoT platform services (Cayenne, ThingSpeak, etc) Code reference: https://stackoverflow.com/questions/3135569

bersim commented 6 years ago

Hi! You need binary library? I changed file for subscribe packet. I see you need also for publish packet.

I some of these days add changing for publish packet.

2017-12-05 15:40 GMT+05:00 Ketut P. Kumajaya notifications@github.com:

For compatibility to some IoT platform services (Cayenne, ThingSpeak, etc) Code reference: https://stackoverflow.com/questions/3135569

You can view, comment on, or merge this pull request online at:

https://github.com/bersim/OpenKPs/pull/5 Commit Summary

  • Force use period (.) as decimal separator

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/bersim/OpenKPs/pull/5, or mute the thread https://github.com/notifications/unsubscribe-auth/AKTLbW65h2_lRjVFHLkUHGK-EMPgfMAOks5s9R2MgaJpZM4Q2E9_ .

kumajaya commented 6 years ago

I have my own binary with patched source but official build much be better.

bersim commented 6 years ago

I builded new version. Can you check it?

https://github.com/bersim/OpenKPs/releases/download/5.0.8/KpMQTT.7z

I added new option for publish topic.

<Topic TopicName="/mesparam10" QosLevel="0" NumCnl="203" PubBehavior="OnChange" Retain="false" NDS="."/>

2017-12-05 19:33 GMT+05:00 Ketut P. Kumajaya notifications@github.com:

I have my own binary with patched source but official build much be better.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/bersim/OpenKPs/pull/5#issuecomment-349321522, or mute the thread https://github.com/notifications/unsubscribe-auth/AKTLbWJQcpOe3BHdXCwzMZzQV4oQQB_yks5s9VQ-gaJpZM4Q2E9_ .

kumajaya commented 6 years ago

I'll test it tomorrow. Thank you.

kumajaya commented 6 years ago

Works as expected. Thank you.