search

kotelnik / plasma-applet-weather-widget

Plasma 5 applet for displaying weather information from yr.no server.
GNU General Public License v2.0
88 stars 42 forks source link

OpenWeatherMap not as nice as yr.no #11

Open frapell opened 8 years ago

frapell commented 8 years ago

yr.no: http://picpaste.com/pics/weather_yrno.1451914961.png owm: http://picpaste.com/pics/weather_owm-rMd0JYmR.1451915177.png

frapell commented 8 years ago

Also, time in widget graph should be local time, not UTC

kotelnik commented 8 years ago

Thanks a lot for all these bug reports. I'll address them as soon as possible.

The city id is ugly, please rename it (as workaround) by clicking the city id in the third column in the list of places. I should also add headers to the table... Eventually I'll add a picker so this should get automated.

What font are you using? Because I've tested only with default fonts with various sizes and DPI and it always looked better (smaller literals which actually fitted in the widget).

The timeline should always start from "now" so there shouldn't be any need of blue line. But first I need to fix the UTC/local-time issue :)

frapell commented 8 years ago

Oh! I was clicking the left number to rename it, didn't try on the right one... it does work, maybe it needs to be a bit more explicit.

As per the fonts, I'm using the default by Kubuntu 15.10, Oxygen-Sans size 8, 9 and 10 (not sure which one is used inside widgets... I change them all but I don't see it changes in the widget)

As per the timeline starting in "now", that's correct, didn't realize that until a couple of hours went by...

As you can see from the screenshot, the time is basically unreadable, being all mashed up together.

kotelnik commented 8 years ago

Unfortunately I'm still unable to replicate the issue with the overlapping (too big) font in rendered meteogram. I tried changing font to Oxygen-Sans, various sizes, changing DPI and also different screen resolutions. Meteogram stayed readable, no font overlapping. (You need to restart plasma in order to apply font changes). I'm suspecting the widget has some non-updated files (like e.g. Meteogram.qml), How did you install it? Is it installed only once? Can you reinstall or ensure that it is up to date? Here is how it should display: ww-github The rest of your reports will be fixed of course :)

frapell commented 8 years ago

I installed it from the "Get new widgets" button from the list of widgets... Now, it seems weird because when I go in there now, I don't see the "Uninstall" option for the widget...

Anyway, I removed the widget, installed it, uninstalled it, and re-installed it again.

Removed the 2 cities that come by default, and added mine (Cordoba, Argentina) from owm. Still seeing it wrong :(

Maybe I need to install some font or something?

kotelnik commented 8 years ago

Thanks a lot for all these tests!

The missing uninstall option is a known plasma bug: https://bugs.kde.org/show_bug.cgi?id=355011 ...well, you can add there that "uninstall" option is therefore also missing if you like :)

You don't need to install anything. I have a suspicion what could cause the different font size... In the widget code I am setting font size with pixels but PlasmaComponent.Label (which I use for showing texts) is already setting pointSize and maybe in Qt 5.4 (kubuntu default) is handling it differently. In Qt 5.5 pixelSize is preferred. I'll post a new version soon and let you know.

kotelnik commented 8 years ago

I tried to blindly address the issue :) Can you please test it before I publish a new version?

After git clone you can build it according README here on github. Or simpler way: call ./create-plasmoid-package.sh to prepare .plasmoid file (it is basically a zip archive with .plasmoid extension) - it will be generated in build directory. This file you can then drop on desktop to install it.

gbcox commented 8 years ago

I've never had much luck using owm - their search function is difficult to use and when you do get results they are often inaccurate. Have you considered including wunderground.com? yr.no appears to be a good service... is also fairly easy to search on and the results I have found to be accurate.

kotelnik commented 8 years ago

You're right, owm is sometimes inaccurate or just not responding. And yes, I've already considered other providers. It just takes time to implement it but I'll eventually get to work on it. I was planning weather.com, yahoo, accuweather... But wunderground.com looks like a paid service. Or do they have also a weak spot for FOSS you know of? :-)

gbcox commented 8 years ago

The payment is optional, and basically just gets rid of the ads. They have some really nice graphics. They do charge for their api, over a certain amount of queries - but that is no different than any of the other services - including OWM. Regarding OWM, I've sent them feedback regarding the inaccuracy of their data. So far, no reply.