5927
93%

Ran 04 Apr 2016 11:00PM UTC

Jobs 4

Files 132

Run time 2min

Badge

Embed ▾

pending completion

Build # 5927

Build Type

Pull #1710

travis-ci

Committed by

JshWright

Commit Message

Use whole degrees fahrenheit for thermostats

Users of fahrenheit generally expect to see whole degrees. The fahreneit
scale is suffiently precise that decimals aren't really useful in terms
of temperatures humans care about.

This change rounds fahrenheit values to whole degrees and celsius values
to one decimal place. It also renames the ThermostatDevice._convert()
method to _convert_for_display(), making its purpose more clear. It is
not useful for the min_temp() and max_temp() methods, as those relate to
the internal state of the ThermostatDevice object, and may use different
units. Adding optional source and target units to _convert() would have
added needless complexity, it's cleaner to just use convert() in those
methods.

Pull Request Pull Request #1710: Use whole degrees fahrenheit for thermostats

Run Details

8987 of 9659 relevant lines covered (93.04%)

1.86 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
1	93.78	-0.52%	homeassistant/components/api.py
1	87.66	-0.43%	homeassistant/remote.py
5	96.24	0.09%	homeassistant/components/thermostat/__init__.py

Jobs

ID	Job ID	Ran	Files	Coverage
1	5927.1 (TOXENV=py34)	04 Apr 2016 11:01PM UTC	132	93.04	Travis Job 5927.1
2	5927.2 (TOXENV=requirements)	04 Apr 2016 11:00PM UTC	0		Travis Job 5927.2
3	5927.3 (TOXENV=lint)	04 Apr 2016 11:03PM UTC	0		Travis Job 5927.3
4	5927.4 (TOXENV=py35)	04 Apr 2016 11:01PM UTC	132	93.04	Travis Job 5927.4

Source Files on build 5927

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

balloob / home-assistant / 5927
93%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 5927

balloob / home-assistant / 5927 93%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 5927

balloob / home-assistant / 5927
93%

README BADGES
x