Skip to content

this commit addresses #65, by adding a preemptive pruner approach: if…#149

Draft
eanokian wants to merge 2 commits intodevfrom
65_count_pruner_efficiency
Draft

this commit addresses #65, by adding a preemptive pruner approach: if…#149
eanokian wants to merge 2 commits intodevfrom
65_count_pruner_efficiency

Conversation

@eanokian
Copy link
Collaborator

@eanokian eanokian commented Jan 13, 2026

Pull Request

Fixes #65

Checks

  • (Have you updated the changelog.md ?)

… we are in a ContentRow (parent row) that contains all zeros, do not go deeper in the TableTree
@eanokian eanokian added the enhancement New feature or request label Jan 13, 2026
@eanokian eanokian self-assigned this Jan 27, 2026
@gmbecker
Copy link
Collaborator

Not sure if this is still a live PR, if not please close it @eanokian. If it is, please handle merge requests and add tests to ensure behavior stays the same

@eanokian
Copy link
Collaborator Author

eanokian commented Mar 17, 2026

@gmbecker it is a live PR, it's a topic I investigated in Jan but I'd like to discuss with you. We are in a point where we need to decide whether to incorporate this minor fix to dev or discard it all along.
please look at the issue #65 for context.
Not urgent though and can be discussed next week.

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.

2 participants