1
96%
master: 93%

Ran 18 Jan 2020 08:10PM UTC

Files 42

Run time 2s

Badge

Embed ▾

Committed 18 Jan 2020 08:07PM UTC coverage: 96.346%. First build

Job # 259.1

Build Type

push

travis-ci

Committed by Chris Mackey

Commit Message

feat(hash): Add __hash__ methods that agree with __eq__ to all objects

I realized that we were overwriting the __eq__ method on a number of objects without overwriting the __hash__ method, which I have come to realize is strongly discouraged in Python. So this commit fixes this by adding hash methods that agree with the __eq__ operator.  This should also allow us to create sets of these geometry objects if we so desire.

The only thing I didn't implement that _might_ be worthwhile to implement in the future is that I am not caching the hash value once it is calculated, which is something that we could do given that all of the geometry objects are immutable. This will allow for quicker hashing if we plan to hash the object multiple times but it also uses up a little more memory per object to store the hash. Right now, we really don't end up hashing the object multiple times in any of our typical workflows such that I think the trade-off isn't worth it. However, we should reconsider this if we find cases in the future where we are hashing the objects multiple times.

Run Details

8306 of 8621 relevant lines covered (96.35%)

0.96 hits per line

ladybug-tools / ladybug-geometry / 259 / 1
96%
master: 93%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 259.1

ladybug-tools / ladybug-geometry / 259 / 1 96% master: 93%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 259.1

ladybug-tools / ladybug-geometry / 259 / 1
96%
master: 93%

README BADGES
x