10 days preview not working

Bug #1588997 reported by Georges Thill
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical System Image
Confirmed
Undecided
Kyle Nitzsche
Ubuntu Rest Scopes
Confirmed
Undecided
Alejandro J. Cura
Ubuntu Weather App
Invalid
Undecided
Unassigned
canonical-scopes-project
Opinion
Undecided
Unassigned

Bug Description

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

Revision history for this message
Georges Thill (gthill) wrote :
Changed in ubuntu-weather-app:
status: New → Invalid
Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

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

Changed in canonical-devices-system-image:
status: New → Incomplete
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

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

Revision history for this message
Georges Thill (gthill) wrote :

Please find the logs you asked for attached.

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

Revision history for this message
Georges Thill (gthill) wrote :

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

Changed in canonical-devices-system-image:
assignee: nobody → Kyle Nitzsche (knitzsche)
status: Incomplete → Confirmed
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

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.

Changed in ubuntu-rest-scopes:
assignee: nobody → Alejandro J. Cura (alecu)
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

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

Changed in ubuntu-rest-scopes:
status: New → Confirmed
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

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.

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

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.

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

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.

Changed in canonical-scopes-project:
status: New → Opinion
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

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.

Revision history for this message
Georges Thill (gthill) wrote :

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

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

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

Revision history for this message
Georges Thill (gthill) wrote :

Thanks Kyle, looking forward to the release!

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.