XBMC Community Forum
[RELEASE] Weather Plus Addon - Printable Version

+- XBMC Community Forum (http://forum.xbmc.org)
+-- Forum: Help and Support (/forumdisplay.php?fid=33)
+--- Forum: Add-ons Help and Support (/forumdisplay.php?fid=27)
+---- Forum: Weather Add-ons (/forumdisplay.php?fid=155)
+---- Thread: [RELEASE] Weather Plus Addon (/showthread.php?tid=95329)



- macf1an - 2011-11-19 10:36

Just got 2.5.0 and unless you manually go with the location selector on every one of the three options in settings - you get "location not selected" message.
Why doesn't it get the proper locations from the global area codes as before?

And the top-of-the-screen temperature indicator from confluence is still gone..


- m4x1m - 2011-11-19 13:47

macf1an Wrote:Just got 2.5.0 and unless you manually go with the location selector on every one of the three options in settings - you get "location not selected" message.
Why doesn't it get the proper locations from the global area codes as before?

And the top-of-the-screen temperature indicator from confluence is still gone..

Same here.

Pre-Eden PVR version doesn't have a git number. Git number is UNKNOWN and this break the script.


- brightsr - 2011-11-19 14:40

olympia Wrote:Is there any reason why you are not requesting this to be included in the official repo?

I'll do when pre-eden issue is gone.


- brightsr - 2011-11-19 15:17

macf1an Wrote:Just got 2.5.0 and unless you manually go with the location selector on every one of the three options in settings - you get "location not selected" message.
Why doesn't it get the proper locations from the global area codes as before?

And the top-of-the-screen temperature indicator from confluence is still gone..

You prefer to the past way. I was thinking all options and area codes should be in the addon's settings as the built-in weather api has been gone. But if users don't like it and want to go back, I'll make a version for Dharma only.


- brightsr - 2011-11-19 15:20

m4x1m Wrote:Same here.

Pre-Eden PVR version doesn't have a git number. Git number is UNKNOWN and this break the script.

Oh, I didn't consider PVR version.
Is the build version like "PRE-11.0 Git:UNKNOWN"?


- Big_Noid - 2011-11-19 15:26

Just reporting in that 2.5 works flawlessly for me on eden-pre.
Only one very very very minor thing: current condition capitalizes also the first letter in the second word. I know accuweather provides this in all capitals so there must be some conversion in the script to lowercase.


- m4x1m - 2011-11-19 15:30

brightsr Wrote:Oh, I didn't consider PVR version.
Is the build version like "PRE-11.0 Git:UNKNOWN"?

unfortunately yes!

I bypassed the check, and I get the "location not selected" message, but latest PVR build is compiled on November 12, I think I'll have to wait a new build to have weather work correctly.


- olympia - 2011-11-19 15:34

brightsr Wrote:I'll do when pre-eden issue is gone.

I don't think there is a pre-eden issue... It work flawless for me on pre-eden and others reported this as well above. The rest is user error.


- brightsr - 2011-11-19 15:38

Big_Noid Wrote:Only one very very very minor thing: current condition capitalizes also the first letter in the second word. I know accuweather provides this in all capitals so there must be some conversion in the script to lowercase.

hmm.. strange.. I already made it to convert current condition to titlecase. (only the first letter is uppercase)


- brightsr - 2011-11-19 16:06

olympia Wrote:I don't think there is a pre-eden issue... It work flawless for me on pre-eden and others reported this as well above. The rest is user error.

I'd like to see if there's no issue until tonight, and then I'll request.