2011-11-19, 16:12
brightsr Wrote:I'd like to see if there's no issue until tonight, and then I'll request.
No problems for me using todays windows build and W+ 2.5
brightsr Wrote:I'd like to see if there's no issue until tonight, and then I'll request.
brightsr Wrote:hmm.. strange.. I already made it to convert current condition to titlecase. (only the first letter is uppercase)
m4x1m Wrote: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.
Big_Noid Wrote:it converts to titlecase okay and that's okay for the first word, but looks odd with the second word because the rest of the days conditions don't do this.
ie Mostly Sunny for today and Mostly sunny for tomorrow.
As I said...very minor issue and if it can't be fixed that's no real issue.
brightsr Wrote: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.
macf1an Wrote:I'm using 10.0-9 which is the last stable eden-pre - not dharma.
Why don't you just check if the guisettings has the area codes set (dharma and all nightlies before the weather api change) and get that info from there?
ERROR: Error Type: exceptions.ValueError
ERROR: Error Contents: invalid literal for int():
ERROR: Traceback (most recent call last):
File "/home/brian/.xbmc/addons/weather.weatherplus/default.py", line 40, in ?
__ver__ = _check_compatible()
File "/home/brian/.xbmc/addons/weather.weatherplus/default.py", line 30, in _check_compatible
ok = (0, 1)[ xbmc_build.startswith("PRE-11") ] * ( ( int( " ".join( re.findall("Git[:]([0-9]+)", xbmc_build) ) ) >= 20111116 ) + 1 )
ValueError: invalid literal for int():
olympia Wrote:Is there any reason why you are not requesting this to be included in the official repo?
brightsr Wrote:[UPDATE] v.2.5.1
- Changed : Compatibilty Check Method ( Checking Version -> Checking Arguments passed to addon )
- Changed : [Builds Before Weather API Change] Using Area Codes Set at Global Weather Settings for Weather.com
- Fixed : [Accuweather.com] Loop Bug
- Improved : [Accuweather.com] Outlooks in title case
* Moving to official repo : I've sent pull request to official repo of dharma and eden-pre. I hope it will be visible soon.
* zip file download : http://xbmc-addons.googlecode.com/files/..._2.5.1.zip
brightsr Wrote:[UPDATE] v.2.5.1
- Changed : Compatibilty Check Method ( Checking Version -> Checking Arguments passed to addon )
- Changed : [Builds Before Weather API Change] Using Area Codes Set at Global Weather Settings for Weather.com
- Fixed : [Accuweather.com] Loop Bug
- Improved : [Accuweather.com] Outlooks in title case
* Moving to official repo : I've sent pull request to official repo of dharma and eden-pre. I hope it will be visible soon.
* zip file download : http://xbmc-addons.googlecode.com/files/..._2.5.1.zip
brightsr Wrote:As built-in weather doesn't work anymore, if you're not using the latest nightly build, 'Weather.IsFetched' is always FALSE regardless weather is fetched by a weather addon. That's why there's no weather icon and temperature on home screen. So you can make them visible by removal of 'Weather.IsFetched' from your skin's xml. If you'd like them to be displayed only after current weather is fetched successfully, you can put '!IsEmpty(Window(Weather).Property(36Hour.IsFetched))' instead of that. ( current weather is fetched during fetching 36 hour forecast information. )
brightsr Wrote:As built-in weather doesn't work anymore, if you're not using the latest nightly build, 'Weather.IsFetched' is always FALSE regardless weather is fetched by a weather addon.