Skip to content

Keep a value in subtract when the other side only passes through - #107

Open
imlvts wants to merge 2 commits into
Adam-Vandervorst:masterfrom
imlvts:fix/subtract-keeps-value
Open

imlvts wants to merge 2 commits into
Adam-Vandervorst:masterfrom
imlvts:fix/subtract-keeps-value

Conversation

@imlvts

@imlvts imlvts commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

ByteNode::psubtract_abstract dropped a value at a byte where the other node has no value, only a longer path. The value now survives.

imlvts and others added 2 commits September 16, 2026 22:27
ByteNode::psubtract_abstract dropped a value at a byte where the other
node has no value, only a longer path. The value now survives.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019R2H8fnco29asY2v3TPbtF
Against a list or tiny node, psubtract_abstract dropped a dangling slot
from the result but left the identity flag set, so the caller kept the
destination and its dangling path.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019R2H8fnco29asY2v3TPbtF
@imlvts

imlvts commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator Author

https://github.com/Adam-Vandervorst/PathMap/actions/runs/35163010909#summary-105018138331

Benchmark: +57.2% superdense_drop_bench/200, but overall results are flat?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant