Skip to content

fix(trie): implement get_values_shallow and get_submaps_shallow for trie #459

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

Merged
merged 1 commit into from
Mar 19, 2022

Conversation

chentoast
Copy link
Contributor

fixes #444. I'm not that familiar with the codebase yet, so please let me know if the changes should go somewhere else, or if this is not the correct fix. Thanks!

@femtomc
Copy link
Contributor

femtomc commented Mar 16, 2022

Re: @chentau -- as per #444, do you mind removing the implementations and the corresponding annotation on Trie (?) I don't want to prevent you from having a PR merged (having already put work into it!) -- so I'd like to give you the option to make these changes on your PR.

If you don't have time to do this, I can do so and push some commits to your PR.

@chentoast chentoast force-pushed the choicemap_tree_impl branch from 0e43442 to d01e827 Compare March 18, 2022 21:18
@chentoast chentoast force-pushed the choicemap_tree_impl branch from d01e827 to a16615f Compare March 18, 2022 21:28
@femtomc
Copy link
Contributor

femtomc commented Mar 18, 2022

Thank you Tony!

@femtomc femtomc merged commit fc75bba into probcomp:master Mar 19, 2022
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.

Gen.Trie does not implement get_values_shallow, despite being a ChoiceMap
2 participants