2
91%
master: 91%

Ran 10 Jul 2020 08:04PM UTC

Files 54

Run time 15s

Badge

Embed ▾

Committed 10 Jul 2020 07:59PM UTC coverage: 91.818% (-0.02%) from 91.833%

Job # 1042.2

Build Type

push

travis-ci

Committed by

chriswmackey

Commit Message

perf(analysisperiod) Improve performance of AnalysisPeriod datetimes

This commit includes a number of performance improvements to the analysis period and (by extension) the hourly continuous data collection.  The biggest improvement comes from the fact that we can get the length of the analysis period without having to create all of the individual datetime objects (we just look at the time difference between the start and end datetimes). Also, when a case arrives that datetimes are needed, these datetimes get cached on the AnalysisPeriod object instead of regenerating them every time that they are called. This should dramatically cut down on the number of datetime objects that we are storing in memory at a given time.

The improvement is good enough that I can even notice the tests running faster.

Technically, there's also a bug fix in this commit but it affects such a niche edge case (reversed analysis periods for leap years) that I don't think it's worth a whole new release right now.

Run Details

7496 of 8164 relevant lines covered (91.82%)

0.92 hits per line

ladybug-tools / ladybug / 1042 / 2
91%
master: 91%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1042.2

ladybug-tools / ladybug / 1042 / 2 91% master: 91%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1042.2

ladybug-tools / ladybug / 1042 / 2
91%
master: 91%

README BADGES
x