10 days preview not working

[Georges Thill]

When I pick one of the 3 weather preview icons (see left part of screenshot in attachement) from "nearby" or "today" scope and then I pick the 10 days preview, I get an error message from weather.com. (see right side of attached screenshot)

- My phone is an MX4
- It is running OTA11
- The configured language is german
- An actual location when this happens is
  visible in left part of screenshot

[Jean-Baptiste Lallement (jibel)]

Thanks for your report. If you can reproduce could you please attach the following files to this bug report:
- .cache/upstart/scope-registry.log
- .cache/upstart/smart-scopes-proxy.log
- .cache/upstart/unity8-dash.log

[Kyle Nitzsche (knitzsche)]

The Weather scope is part of the Ubuntu Rest Scope project.

[Georges Thill (gthill)]

Please find the logs you asked for attached.

In this location the problem is is *always* easy to reproduce.

[Georges Thill (gthill)]

2 months have passed. Is there any news on this problem?

[Kyle Nitzsche (knitzsche)]

I believe the unity-scopes team owns the ubuntu-rest-scopes, of which weather scope is part.

I suppose this should be assigned to alecu.

[Kyle Nitzsche (knitzsche)]

This bug was encountered again and was again filed against the Today scope :) I've marked it a dup of this.

[Kyle Nitzsche (knitzsche)]

I noticed from the logs that when the url used to open the 10 day forecast on the weather channel website does not have latitude and longitude with two digits after the decimal point, weather channel reports an issue.

For example, the log from the bug I just marked a dup shows this:
https://weather.com/weather/10day/l/47,8?par=ubuntu_bq

A failure. But if you simply add the required precision like this:
https://weather.com/weather/10day/l/47.00,8.00

The 10 day forecast web page loads successfully.

[Kyle Nitzsche (knitzsche)]

In the logs.tar.gz attached to this bug in comment #4, I see this:
https://weather.com/de-DE/weather/today/l/49.6,6.13?par=ubuntu

Which fails.

If you remove de-DE AND make the latitude double precision, like this, it succeeds:
https://weather.com/weather/today/l/49.61,6.13

ONLY going double precision on the lat/long is not enough.

So it seems there is ALSO an issue with the usage of the localization ("de-DE") in the URL.

[Kyle Nitzsche (knitzsche)]

A workaround for the lack of double precision issue would probably be to check the precision of the lat & long and pad with zeros if it is too short.

The localization issue would probably required looking at the TWC rest API and figuring out if the ubuntu rest scope is doing something wrong.

[Kyle Nitzsche (knitzsche)]

Since there are apparently two issues (lack of precision in forecast url lat&long and locale param in url), I am unduplication the previous bug, which will handle lat&long issues, for which I will have a mergre proposal shortly.

This bug can handle the url locale param issue.

[Georges Thill (gthill)]

Any news on this? So far I still encounter this issue.

[Kyle Nitzsche (knitzsche)]

Hi George,

I've submitted a merge proposal to fix for the lat/long issue, and it is pending review:

bug: https://bugs.launchpad.net/ubuntu-rest-scopes/+bug/1629243
merge proposal: https://code.launchpad.net/~knitzsche/ubuntu-rest-scopes/fix-10-day-forecast-url/+merge/307347

[Georges Thill (gthill)]

Thanks Kyle, looking forward to the release!