Closed dglmoore closed 5 years ago
Merging #165 into master will increase coverage by
1.07%
. The diff coverage is98.86%
.
@@ Coverage Diff @@
## master #165 +/- ##
==========================================
+ Coverage 87.34% 88.41% +1.07%
==========================================
Files 16 16
Lines 1470 1485 +15
==========================================
+ Hits 1284 1313 +29
+ Misses 186 172 -14
Impacted Files | Coverage Δ | |
---|---|---|
neet/network.py | 64% <ø> (-0.29%) |
:arrow_down: |
neet/boolean/reca.py | 82.64% <100%> (+0.29%) |
:arrow_up: |
neet/boolean/eca.py | 97.14% <100%> (+0.08%) |
:arrow_up: |
neet/synchronous.py | 97.69% <98.79%> (+6.91%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 9b459c4...1b7f2f2. Read the comment docs.
This looks great. I tried to include __landscaped
in the Mixin class originally but I was doing it wrong. Thanks for properly setting up the clear and initialization for landscape
.
This pull request does the following:
LandscapeMixin
internal variables fromNetwork
LandscapeMixin.landscape_data
propertyLandscapeData
LandscapeMixin.__expound
toLandscapeMixin.expound
clear_landscape
method (which resets the internal landscape data)clear_landscape
in theECA
andRewiredECA
property setters