Bug 176337 - LCD weather widget wrong on current weather
Summary: LCD weather widget wrong on current weather
Status: RESOLVED INTENTIONAL
Alias: None
Product: plasma4
Classification: Unmaintained
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-28 14:32 UTC by Kristjan Ugrin
Modified: 2017-02-02 20:00 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kristjan Ugrin 2008-11-28 14:32:16 UTC
Version:           1.0 (using Devel)
OS:                Linux
Installed from:    Compiled sources

When using LCD weather widget with bbcukmet plugin, it will always show rainy weather.
City: ljubljana

I checked the bbc page and it correctly displays forecast - probably a bug inside widget.
Comment 1 Anne-Marie Mahfouf 2008-11-28 14:52:53 UTC
Confirmed, same here while it's a 100% sunny day and indicated as such by BBC
Comment 2 Anne-Marie Mahfouf 2008-11-28 15:33:12 UTC
I looked at the code and the hack on the "p" variable does not work
from line 170 in weatherstation.cpp

Before
    if (p > 103.0) {
        current << "sun";
p is zero.

If you give me some criterias on how you want it to work Petri I can fix it.

Thanks for looking at it as a lot of users complained about it!

Anne-Marie
Comment 3 Aaron J. Seigo 2008-11-28 18:55:21 UTC
it mimics a traditional barometer. it's not meant to show the "actual" weather but the direction of the weather changes (nicer or poorer weather).

the weather applet that is *STILL* in playground is the one that does what people think ;)

since this is intended behaviour and has an actual reason for it, it's a WONTFIX imho.
Comment 4 Anne-Marie Mahfouf 2008-11-28 19:08:11 UTC
I disagree. The string displays "Current weather" above the picture of the weather. In my opinion it's just a matter of fixing the way the pic is shown. 

A lot of users have been puzzled by this behavior and this is a popular applet. 

I'll fix it.
Comment 5 Aaron J. Seigo 2008-11-28 19:49:03 UTC
the string "current weather" could be changed perhaps, but i really don't think changing the intended purpose of this widget is desirable.

the weather widget does it "right" because that's what it is meant to do.
Comment 6 Kristjan Ugrin 2008-11-28 19:52:20 UTC
It doesn't do anything right - even if it is acting like a barometer it's doing it wrong.
It always shows rainy weather, even when the parameters are changing from bad to good.
Comment 7 Anne-Marie Mahfouf 2008-11-29 11:57:56 UTC
SVN commit 890430 by annma:

Hectopascals is not in WeatherUtils::convert srcDir so the BBC pressure was converted 
from something that does not exist hence the always raining pic.
Now should behave like a barometer, meaning that if it's raining but the tendency is "Rising" the pic can show a sun. This calculation though could be tweaked further.
CCBUG=176337


 M  +2 -2      weatherstation.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=890430
Comment 8 Allan Johnstone 2017-02-02 20:00:29 UTC
I am using the Weather Widget on OpenSUSE Leap 42.2, running plasma5-addons 5.8.3-7.1. The panel display of the weather information has shown -7, -3, and 0 degrees C: the "near future" temperature shown in the mouse-over pop-up is 4 degrees C, which matches the Environment Canada reported temperature (I am using OWM as my weather data source). Ah, the reported temperature just jumped down to -5 C. Also, the precipitation icon has been on rainfall all day, yet today is brilliantly clear and sunny. Again, the mouse-over pop-up is accurate... oops, the mouse-over temperature is now showing -1 degree C, yet it is still 4 degrees C. on the Environment Canada website listing for Vancouver. These rapid changes and inaccuracies are clearly in error.