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

feat(Trie): map_tag #132

Merged
merged 2 commits into from
Jul 5, 2024
Merged

feat(Trie): map_tag #132

merged 2 commits into from
Jul 5, 2024

Conversation

favonia
Copy link
Collaborator

@favonia favonia commented Jun 18, 2024

Close #124.

@favonia
Copy link
Collaborator Author

favonia commented Jun 18, 2024

@mikeshulman Not sure if you still want it but I am clearing the backlog...

@mikeshulman
Copy link

Hmm, at the moment I don't even remember what I wanted this for. But maybe it'll come to me. (-:

@favonia
Copy link
Collaborator Author

favonia commented Jun 19, 2024

@mikeshulman I vaguely remember it's about using tags to remember precedence and this function can update all precedence at once. Maybe I should also include a version for subtrees...

@mikeshulman
Copy link

Ah, yes, I found the old email thread.

@mikeshulman
Copy link

I thought of something else I might also use this for. And the PR looks good to me, if you're amenable to merging it.

@favonia favonia merged commit 2c48886 into main Jul 5, 2024
1 check passed
@favonia favonia deleted the map_tag branch July 5, 2024 18:05
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.

🔖 Add Trie.map_tag?
2 participants