Skip to content

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

Merged
merged 21 commits into from
Jan 28, 2020
Merged

Lora support #83

merged 21 commits into from
Jan 28, 2020

Conversation

fabik111
Copy link
Contributor

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
Contributor 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