Closed eugeniashurko closed 3 years ago
Merging #87 (75535e0) into master (e41b415) will increase coverage by
0.08%
. The diff coverage is83.33%
.
@@ Coverage Diff @@
## master #87 +/- ##
==========================================
+ Coverage 81.16% 81.25% +0.08%
==========================================
Files 41 44 +3
Lines 3680 3723 +43
==========================================
+ Hits 2987 3025 +38
- Misses 693 698 +5
Impacted Files | Coverage Δ | |
---|---|---|
bluegraph/backends/graph_tool/io.py | 84.54% <ø> (-0.07%) |
:arrow_down: |
bluegraph/core/analyse/metrics.py | 73.58% <ø> (ø) |
|
bluegraph/downstream/similarity.py | 92.20% <ø> (-0.06%) |
:arrow_down: |
bluegraph/downstream/utils.py | 88.00% <ø> (ø) |
|
bluegraph/preprocess/encoders.py | 79.18% <ø> (ø) |
|
bluegraph/backends/neo4j/analyse/metrics.py | 72.09% <50.00%> (ø) |
|
bluegraph/backends/utils.py | 71.42% <71.42%> (ø) |
|
bluegraph/backends/configs.py | 79.31% <79.31%> (ø) |
|
bluegraph/backends/graph_tool/analyse/paths.py | 92.98% <100.00%> (ø) |
|
bluegraph/backends/neo4j/embed/embedders.py | 90.82% <100.00%> (-0.17%) |
:arrow_down: |
... and 11 more |
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 e41b415...75535e0. Read the comment docs.
bluegraph.backends.utils
).For example, instead of doing the following:
(which includes a lot of backend-specific imports) user can also do:
The same holds for node embedders, for example:
Separated backend configs and node embedding model params into separate config files (adding new backends and models made easier)
Bugfix in
watchyourstep
embedding model ofStellarGraphNodeEmbedder
: lost mandatory parametersteps_per_epoch
Cleanup in Glucose notebook: removed lost test cell, cleanup in
bluegraph
's code: removed unused vars