-
Notifications
You must be signed in to change notification settings - Fork 242
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
Trie should not be specific to subclasses of AbstractString
#220
Comments
@oxinabox, care to take a stab at implementing this? |
I could probably look into this. @oxinabox I'm not very familiar with the term "free monoid" and the wikipedia entry is a bit dense, but would that include |
Yeah, strings, but symbols (except if you convert them to strings). Basically a try can be indexed with I'm in no hurry for this, it is more of a nice-to-have. |
@oxinabox Alright, I have a prototype of supporting both |
Tries work find for any collection of elements of a free monoid (I think I have that term right).
Not just strings,
but also things like arrays of Ints.
Technically arrays of Float64s but that causes other problems.
Wikipedia gets several examples on nonstrings.
I haven't have a close look at the code, but it seems like this is simply a matter of relaxing a type constraint.
Particularly since these are uncompressed dict based Tries.
The text was updated successfully, but these errors were encountered: