Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix unit test to work with new polars #321

Merged
merged 3 commits into from
Nov 5, 2024
Merged

Fix unit test to work with new polars #321

merged 3 commits into from
Nov 5, 2024

Conversation

kayjan
Copy link
Owner

@kayjan kayjan commented Nov 5, 2024

Description

Testing

Additional notes

Checklist

I have read through the contributing guidelines and ensured that

  • I have added a descriptive title for this pull request.
  • I have followed the convention and standards, and my code is checked for style and correctness.
  • I have added test cases, and unit tests pass with 100% code coverage.
  • I have updated the documentation and code docstrings.

Checklist (for reviewer)

  • I have added label (breaking / enhancement / bug / documentation) to this pull request, if applicable.
  • I will ensure this change is captured in the CHANGELOG.md file.

@kayjan kayjan added the enhancement New feature or request label Nov 5, 2024
Copy link
Contributor

github-actions bot commented Nov 5, 2024

Code Coverage

Package Line Rate Branch Rate Complexity Health
bigtree 100% 100% 0
bigtree.binarytree 100% 100% 0
bigtree.dag 100% 100% 0
bigtree.node 100% 100% 0
bigtree.tree 100% 100% 0
bigtree.utils 100% 100% 0
Summary 100% (2388 / 2388) 100% (1006 / 1006) 0

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark

Benchmark suite Current: 7eab9c3 Previous: d0dfa7c Ratio
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_10 15100.493786235735 iter/sec (stddev: 0.00001835267366365161) 16046.316086585764 iter/sec (stddev: 0.000003435831852204441) 1.06
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_100 686.9023756140327 iter/sec (stddev: 0.000005324584773693746) 694.2359567083926 iter/sec (stddev: 0.0000079456881890127) 1.01
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_1000 11.86230610397871 iter/sec (stddev: 0.0016994779741523914) 11.820206885038557 iter/sec (stddev: 0.0012306550059370468) 1.00
tests/node/test_node_benchmark.py::test_node_benchmark_width_2_depth_10 62.53741320287627 iter/sec (stddev: 0.004341735409252626) 68.17562713762312 iter/sec (stddev: 0.0034786572770125605) 1.09
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_10_no_assertions 22374.067560186784 iter/sec (stddev: 0.000008727819000337222) 11148.601380022468 iter/sec (stddev: 0.00006557460801318532) 0.50
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_100_no_assertions 1737.145708046884 iter/sec (stddev: 0.0000037751723941760654) 1774.1265464920377 iter/sec (stddev: 0.000004787607883342527) 1.02
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_1000_no_assertions 110.77870422315 iter/sec (stddev: 0.003914686329771476) 119.4883925241292 iter/sec (stddev: 0.003140809010873645) 1.08
tests/node/test_node_benchmark.py::test_node_benchmark_width_2_depth_10_no_assertions 87.03601778885432 iter/sec (stddev: 0.004417919659543119) 92.00919677925535 iter/sec (stddev: 0.0035359704029477505) 1.06

This comment was automatically generated by workflow using github-action-benchmark.

@kayjan kayjan merged commit e9ac904 into master Nov 5, 2024
6 checks passed
@kayjan kayjan deleted the feature/new-polars branch November 5, 2024 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant