Closed YamLyubov closed 1 year ago
Hello @YamLyubov! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Merging #212 (5298ef2) into main (2f2df8d) will decrease coverage by
0.01%
. Report is 4 commits behind head on main. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #212 +/- ##
==========================================
- Coverage 73.76% 73.76% -0.01%
==========================================
Files 131 131
Lines 7826 7833 +7
==========================================
+ Hits 5773 5778 +5
- Misses 2053 2055 +2
Files | Coverage Δ | |
---|---|---|
golem/core/adapter/nx_adapter.py | 81.66% <ø> (-0.31%) |
:arrow_down: |
golem/core/optimisers/genetic/gp_operators.py | 92.20% <100.00%> (+0.54%) |
:arrow_up: |
golem/core/optimisers/optimizer.py | 95.60% <100.00%> (+0.04%) |
:arrow_up: |
Fixes:
structural_equivalent_nodes
Adds test for evolution with crossover.
206