Skip to content

Conversation

@fabik111
Copy link
Collaborator

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

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 9ae4aa80-1b67-11ea-b9ce-6f43500ed087

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 17420f30-1b6a-11ea-b9ce-6f43500ed087

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 62597b60-1b70-11ea-bf43-1b2acb1c8aac

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: b95df0d0-1b70-11ea-bf43-1b2acb1c8aac

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 98369e40-1c27-11ea-ba59-e12deadb3a4c

Copy link
Contributor

@aentinger aentinger left a 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?

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 2e86ec70-2186-11ea-a044-67124c782b74

Copy link
Collaborator Author

@fabik111 fabik111 left a 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

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 914f2da0-2243-11ea-b76a-c509dfe0ae07

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 05c8c000-2272-11ea-b76a-c509dfe0ae07

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: f20213e0-2272-11ea-b76a-c509dfe0ae07

@TravisBuddy
Copy link

Hey @fabik111,
Something went wrong with the build.

TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.

View build log

TravisBuddy Request Identifier: ba1d4590-2339-11ea-8e2a-07cdc06d0399

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 8de19470-35e7-11ea-b480-9fec542541c3

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: d013de00-3855-11ea-b5ea-ed59922799f5

…rduinoIoTCloud_Travis_CI need to be updated too in order to support MKRWAN boards
@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 165c1aa0-3877-11ea-b5ea-ed59922799f5

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 0fe77c60-392a-11ea-b1da-df51646d6421

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: f860eba0-3936-11ea-b1da-df51646d6421

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 105f4a60-40ff-11ea-a60c-ed450572c3ce

@TravisBuddy
Copy link

Travis tests have failed

Hey @fabik111,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: d0e4d430-4109-11ea-a60c-ed450572c3ce

@aentinger aentinger merged commit 2c34b4c into arduino-libraries:master Jan 28, 2020
@aentinger aentinger deleted the lora-support branch January 28, 2020 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants