Domoticz

fees wrote:I'm not sure whether go combine temperature, humidity and barometer into one or two devices or make three separate devices for that. Not sure what is best practice/most common/most desired.

Snowtiger wrote:

fees wrote:I'm not sure whether go combine temperature, humidity and barometer into one or two devices or make three separate devices for that. Not sure what is best practice/most common/most desired.

My opinion - combine them like Weather Underground does - Temp + Humidity + Baro is a known device type and subtype, if someone wants to separate them - it is always possible to write a lua script which checks change of the device and split them up into separate values

ffes wrote:Thanks you all for spotting the bug in getWindChill(). You are right, it should be self.temperature. When I coded it, it was too cold to find this error

I've committed this fix to the repo. Do "git pull"

error: Your local changes to the following files would be overwritten by merge:
        buienradar.py
Please, commit your changes or stash them before you can merge.
Aborting

Snowtiger wrote:

ffes wrote:Thanks you all for spotting the bug in getWindChill(). You are right, it should be self.temperature. When I coded it, it was too cold to find this error

I've committed this fix to the repo. Do "git pull"

Gitt pull fails - msg -

Code: Select all

error: Your local changes to the following files would be overwritten by merge:
        buienradar.py
Please, commit your changes or stash them before you can merge.
Aborting

my bad - changed the file to correct the error myself

fkok wrote:I had the same, because i changed the file myself. first do a git reset --hard, en then a git pull.

ffes wrote: I also added the code to add the wind device.

EldigoR wrote:

ffes wrote: I also added the code to add the wind device.

Nice work with the Wind device!
I think it would beter to have Wind+Temp+Chill as Weather underground also has

ffes wrote:I just update the github repository. The wind is now a "Wind+Temp+Chill" device and the rounding is improved .

Brutus wrote:I don't understand why you would like to have alle the values in one sensor like WU does. I think the sensors of WU are a mess, I see the temperature on three sensors at the same time and the "Wind" sensor is linked to a seperated temperature sensor. Changing the name for one of the two sensors will change the other one to. Not logical to me.

So for me I hope it gets separated so its clean.

Brutus wrote:I don't understand why you would like to have alle the values in one sensor like WU does. I think the sensors of WU are a mess, I see the temperature on three sensors at the same time and the "Wind" sensor is linked to a seperated temperature sensor. Changing the name for one of the two sensors will change the other one to. Not logical to me.

So for me I hope it gets separated so its clean.

Brutus wrote:I don't understand why you would like to have alle the values in one sensor like WU does. I think the sensors of WU are a mess, I see the temperature on three sensors at the same time and the "Wind" sensor is linked to a seperated temperature sensor. Changing the name for one of the two sensors will change the other one to. Not logical to me.

So for me I hope it gets separated so its clean.

ffes wrote:I think I can make that optional (two combined devices or 5 separate devices). What would make everybody happy!

I would make my life more difficult because the combined temp+hum+baro wants other values for BAR_FOR then the plain Barometer device: https://www.domoticz.com/wiki/Domoticz_ ... #Barometer

ffes wrote:I think I can make that optional (two combined devices or 5 separate devices). What would make everybody happy!

I would make my life more difficult because the combined temp+hum+baro wants other values for BAR_FOR then the plain Barometer device: https://www.domoticz.com/wiki/Domoticz_ ... #Barometer

Snowtiger wrote:

Brutus wrote:I don't understand why you would like to have all the values in one sensor like WU does. I think the sensors of WU are a mess, I see the temperature on three sensors at the same time and the "Wind" sensor is linked to a separated temperature sensor. Changing the name for one of the two sensors will change the other one to. Not logical to me.

So for me I hope it gets separated so its clean.

You can separate it yourself - why 8 or ten values added by one hardware device - to much device sensors which you do not use
And after all , its also a matter of taste ...

Brutus wrote:But I think it's logic that you combine the sensors that are related to each other. For example:

* Temperature
* Wind speed - Wind Direction - Wind Chill - Wind Speed Gust.
* Barometer - Barometer Forecast
* etc

For me I think the separation of the linked temperature sensors is a good way to start

With the latest update we have a separated Temperature sensor, Wind sensor with again a linked temperature sensor you can't delete.

Snowtiger wrote:

Brutus wrote:But I think it's logic that you combine the sensors that are related to each other. For example:

* Temperature
* Wind speed - Wind Direction - Wind Chill - Wind Speed Gust.
* Barometer - Barometer Forecast
* etc

For me I think the separation of the linked temperature sensors is a good way to start

With the latest update we have a separated Temperature sensor, Wind sensor with again a linked temperature sensor you can't delete.

I agree with the logic part - but you have to be sure there is a Device present in Domoticz that can take the values you want.

And for separation of values - keep in mind that wind chill is also a temperature value, the "wgr800 windsensor" earlier used today does not incorporate this value