Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Netatmo binding stops to work once it starts to rain #866

Closed
pail23 opened this issue May 1, 2016 · 10 comments
Closed

Netatmo binding stops to work once it starts to rain #866

pail23 opened this issue May 1, 2016 · 10 comments
Assignees
@clinique

Comments

@pail23
Copy link
Contributor

pail23 commented May 1, 2016 •
edited by kaikreuzer
Loading

I have a netatmo station with a rain module installed. Once it starts to rain, the netatmo binding creates the following exception:
com.google.gson.JsonSyntaxException: java.lang.NumberFormatException: Expected an int but was 6.06 at line 1 column 788 path $.body.modules[1].dashboard_data.sum_rain_24
@clinique
Copy link
Contributor

clinique commented May 2, 2016

@pail23 : second incident raised on this matter. Will take a look very soon.

@clinique
Copy link
Contributor

clinique commented May 2, 2016 •
edited
Loading

Yes, got it. In swagger, rain measures are described as Integer, need to correct this.

@jongj jongj mentioned this issue May 11, 2016
Merged
@clinique
Copy link
Contributor

Was about to correct this, but currently not able to generate new version of swagger client due to opened issue in swagger codegen (swagger-api/swagger-codegen#2096) - will have to wait a bit :(

@clinique clinique mentioned this issue May 24, 2016
Closed
@cbornet
Copy link

cbornet commented May 24, 2016

@clinique I removed the use of allOf in the swagger declaration to workaround the inheritance issue. Can you have a look ?

@clinique
Copy link
Contributor

@cbornet : thanks a lot, I'll go on with this this week-end.

@clinique
Copy link
Contributor

@cbornet : done, it works! Thanks a lot

@clinique
Copy link
Contributor

PR pushed

@clinique clinique mentioned this issue May 24, 2016
Merged
@lolodomo
Copy link
Contributor

lolodomo commented Nov 1, 2016

@clinique : this issue should be closed ?

@clinique
Copy link
Contributor

clinique commented Nov 1, 2016

@lolodomo , it should be closeable while merged #1099

@clinique clinique mentioned this issue Nov 18, 2016
Merged
@lolodomo
Copy link
Contributor

Issue to close.

Flole998 pushed a commit to Flole998/openhab-addons that referenced this issue Dec 30, 2021
@wborn @maggu2810
Fix serial ports missing when ports are added to system property (ope…
f3ced23 
…nhab#866)

* Get all serial port identifiers by calling RXTX with and without using the gnu.io.rxtx.SerialPorts property
* Filter getSerialPortIdentifiers() stream on distinct port names
* Fix thread safety issues by saving getPortIdentifiers() Enumeration result to new list

Fixes openhab#805

Signed-off-by: Wouter Born <github@maindrain.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@clinique clinique

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@kaikreuzer @clinique @lolodomo @cbornet @pail23