Opened 10 years ago

Closed 10 years ago

#8255 closed defect (fixed)

mythweather: wrong wind speed SI unit

Reported by: alec leamas <leamas.alec%at%gmail.com> Owned by: beirdo
Priority: minor Milestone: 0.24
Component: Plugin - MythWeather Version: 0.22-fixes
Severity: medium Keywords: mythweather wind SI from three aspects:
Cc: Ticket locked: no

Description

As of 0.22, mythweather presents wind speed as kph (kilometers per hour?) when user selects SI units. This is problematic:

-"kph" is normally written "km/h".(?)

  • The SI-unit for speed is meters per second("m/s").
  • Users selecting SI units probably expects m/s.

Another issue with wind speed is that many users around the world uses knots and Beaufort, so a simple SI/English units preference is really not enough. Most sw out there displaying wind speed have preferences to display the speed either as mph,knots,Beaufort or m/s.

This might or might not affect the grabber interface depending on how things are implemented. However, in the long run I think SI units should mean just SI units (i. e., m/s) also in the grabber interface.

Change History (4)

comment:1 Changed 10 years ago by alec leamas <leamas.alec%at%gmail.com>

oops, this is on 022-fixes.And definitely mythweather plugin. Can someone please...

comment:2 Changed 10 years ago by beirdo

Component: MythTV - GeneralPlugin - MythWeather
Version: unknown0.22-fixes

I beg to differ. In Canada, wind speeds are km/h, not m/s. km/h are still SI units, just not in base units.

comment:3 Changed 10 years ago by beirdo

Milestone: unknown0.24
Owner: changed from Isaac Richards to beirdo
Status: newassigned

I'll look at changing kph to km/h though.

comment:4 Changed 10 years ago by beirdo

Resolution: fixed
Status: assignedclosed

(In [23992]) Mythweather changes:

  • added "temp" (from 18 day screen) to list of items that gets deg F or deg C added to the value
  • changed kph to km/h - Closes #8255
  • tidied the scripts/README file
  • added "copyright" to every screen as presented by weather-screens.xml (refers to #8253)
  • added UTF-8 support for script output (in particular, -l results, and weather data results)
  • changed the local cache file to replace / with - as the yr.no script uses / in the station_id/location name
  • added the option to return no data for an item by returning "---"
  • increased the size of the location field in weatherdatalayout to varchar(128)
  • bumped mythweather DB schema to 1005

New weather services:

  • added yr.no service - 3 day / 6 day / 18 hour (actually 36 hour)
  • added wunderground service - 3 day / 6 day
Note: See TracTickets for help on using tickets.