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

Tidy and add tests for modify module #334

Merged
merged 5 commits into from
Dec 26, 2024
Merged

Conversation

kayjan
Copy link
Owner

@kayjan kayjan commented Dec 26, 2024

Description

Testing

  • Organise and make tests shorter and standardised
  • Rename self.root_overriding to something more generic
  • Create tests for all shifts (overriding/merge_attribute with delete_children parameter)

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 Dec 26, 2024
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: 415afb7 Previous: 4459ff4 Ratio
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_10 14744.076014540557 iter/sec (stddev: 0.000015894982624666463) 14808.836432609516 iter/sec (stddev: 0.000018276093998188393) 1.00
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_100 688.4677791575135 iter/sec (stddev: 0.000015563420256424255) 690.9404407618113 iter/sec (stddev: 0.000020024486224810658) 1.00
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_1000 11.850073013691683 iter/sec (stddev: 0.0025916774506448245) 11.798482223307227 iter/sec (stddev: 0.0025368368649917607) 1.00
tests/node/test_node_benchmark.py::test_node_benchmark_width_2_depth_10 67.03145227683117 iter/sec (stddev: 0.003500444297600789) 68.32946067573548 iter/sec (stddev: 0.0032854353993074337) 1.02
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_10_no_assertions 22566.46104898742 iter/sec (stddev: 0.000008927576667728774) 22021.12045742032 iter/sec (stddev: 0.000002298167749013623) 0.98
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_100_no_assertions 1766.1840745318093 iter/sec (stddev: 0.000014128134911286635) 1687.8259561135467 iter/sec (stddev: 0.00006115391783906489) 0.96
tests/node/test_node_benchmark.py::test_node_benchmark_width_1_depth_1000_no_assertions 118.7648858052187 iter/sec (stddev: 0.0034249366777931926) 119.16914462020154 iter/sec (stddev: 0.0034163613817203412) 1.00
tests/node/test_node_benchmark.py::test_node_benchmark_width_2_depth_10_no_assertions 94.30026651423036 iter/sec (stddev: 0.002952354376374877) 91.65405548871612 iter/sec (stddev: 0.0033109019914501357) 0.97

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

Copy link
Contributor

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% (2429 / 2429) 100% (1018 / 1018) 0

@kayjan kayjan merged commit cfc1f6f into master Dec 26, 2024
6 checks passed
@kayjan kayjan deleted the feature/tidy-modify-tests branch December 26, 2024 10:24
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