27916
94%

Ran 10 Aug 2017 08:38AM UTC

Jobs 5

Files 280

Run time 4min

Badge

Embed ▾

pending completion

Build # 27916

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

LIFX: improve performance of multi-light transitions (#8873)

* LIFX: improve performance of multi-light transitions

To avoid hub overload, the light.turn_on call will change each light
sequentially.

As LIFX has no hub we can safely increase performance by starting all
light transitions concurrently.

* Improve state updates after light changes

The light.turn_on call will set a new state and then immediately read it
back. However, reading the state of a LIFX light right after a state
change can still return the old value.

To handle this situation we have previously delayed the update request a
little while to allow a potential state change to settle. Because light
updates are now run in parallel, this delay might be too short when many
lights are set at once.

This commit introduces a per-light Lock to make it explicit when the
state cannot yet be trusted.

We must then do the state update ourselves. This was already done at the
end of a long transition and that code can be reused for also doing the
update at the start of a transition.

Run Details

24423 of 26049 relevant lines covered (93.76%)

2.81 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	27916.1 (TOXENV=lint)	10 Aug 2017 08:42AM UTC		Travis Job 27916.1
2	27916.2 (TOXENV=py34)	10 Aug 2017 08:41AM UTC	93.74	Travis Job 27916.2
3	27916.3 (TOXENV=py35)	10 Aug 2017 08:41AM UTC	93.74	Travis Job 27916.3
4	27916.4 (TOXENV=py36)	10 Aug 2017 08:41AM UTC	93.73	Travis Job 27916.4
5	27916.5 (TOXENV=requirements)	10 Aug 2017 08:38AM UTC		Travis Job 27916.5

Source Files on build 27916

Detailed source file information is not available for this build.

home-assistant / home-assistant / 27916
94%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 27916

home-assistant / home-assistant / 27916 94%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 27916

home-assistant / home-assistant / 27916
94%

README BADGES
x