-
Notifications
You must be signed in to change notification settings - Fork 82
Lora support #83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Lora support #83
Conversation
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: 9ae4aa80-1b67-11ea-b9ce-6f43500ed087 |
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: 17420f30-1b6a-11ea-b9ce-6f43500ed087 |
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: 62597b60-1b70-11ea-bf43-1b2acb1c8aac |
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: b95df0d0-1b70-11ea-bf43-1b2acb1c8aac |
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: 98369e40-1c27-11ea-ba59-e12deadb3a4c |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So far this PR looks reasonably good. A couple of questions:
- Why is there no CloudSerial for LoRa (as far as I know its used for the getting started examples? - how does the LoRa board do that?)
- What is the difference between LoRa and WiFi/GSM and does the it justify a separation of the classes?
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: 2e86ec70-2186-11ea-a044-67124c782b74 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Why is there no CloudSerial for LoRa (as far as I know its used for the getting started examples? - how does the LoRa board do that?)
CloudSerial will be not available for LoRa because we have bandwidth limitation and users risk to never send messages with property values but only with logs
* What is the difference between LoRa and WiFi/GSM and does the it justify a separation of the classes?
There are many differences between LoRa and WiFi (p.e. LoRa has not mqtt...), we offer an interface that can be easily inherited for future protocols and can extends Arduino Cloud for other boards
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: 914f2da0-2243-11ea-b76a-c509dfe0ae07 |
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: 05c8c000-2272-11ea-b76a-c509dfe0ae07 |
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: f20213e0-2272-11ea-b76a-c509dfe0ae07 |
Hey @fabik111, TravisCI finished with status TravisBuddy Request Identifier: ba1d4590-2339-11ea-8e2a-07cdc06d0399 |
5563800
to
c2a03b9
Compare
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: 8de19470-35e7-11ea-b480-9fec542541c3 |
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: d013de00-3855-11ea-b5ea-ed59922799f5 |
…rduinoIoTCloud_Travis_CI need to be updated too in order to support MKRWAN boards
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: 165c1aa0-3877-11ea-b5ea-ed59922799f5 |
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: 0fe77c60-392a-11ea-b1da-df51646d6421 |
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: f860eba0-3936-11ea-b1da-df51646d6421 |
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: 105f4a60-40ff-11ea-a60c-ed450572c3ce |
Travis tests have failedHey @fabik111, TravisBuddy Request Identifier: d0e4d430-4109-11ea-a60c-ed450572c3ce |
Support for LoRaWAN boards.
ArduinoIoTCloud has been separated in ArduinoIoTCloudTCP and ArduinoIoTCloudLPWAN handlers to support TCP and Lora connection. ArduinoIoTCloud becomes a pure virtual class.
One of these two object is instantiated according board type