FIX/#199 #209

joocer · 2022-06-18T23:50:11Z

No description provided.

ghost · 2022-06-18T23:51:19Z

👇 Click on the image for a new way to code review

Make big changes easier — review code in small groups of related files
Know where to start — see the whole change at a glance
Take a code tour — explore the change with an interactive tour
Make comments and review — all fully sync’ed with github

Try it now!

Legend

codecov · 2022-06-18T23:51:26Z

Codecov Report

Merging #209 (2ecf32a) into main (ca8d138) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #209   +/-   ##
=======================================
  Coverage   92.23%   92.23%           
=======================================
  Files          91       91           
  Lines        3386     3387    +1     
=======================================
+ Hits         3123     3124    +1     
  Misses        263      263

Impacted Files	Coverage Δ
...pteryx/engine/planner/operations/selection_node.py	`89.81% <ø> (ø)`
opteryx/engine/planner/operations/show_columns.py	`91.35% <100.00%> (ø)`
opteryx/sketches/__init__.py	`100.00% <100.00%> (ø)`
opteryx/version.py	`100.00% <100.00%> (ø)`
tests/sketches/test_counting_tree.py	`58.33% <100.00%> (ø)`

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 ca8d138...2ecf32a. Read the comment docs.