-
Notifications
You must be signed in to change notification settings - Fork 37
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
Port PhraseBuilder from v1.2 branch to master #20
Comments
Rupert, do you already have plans for this or should I get started? |
Doing a git merge would likely be a mess so doing this manually is probably the way to go, except for a few particular cases perhaps where a cherry-pick would probably work (e.g. making TagLL public). It could be done with a sequence of commits adding one feature at a time:
|
Updated TODO list based on changes since the last (removed two items):
|
Updated TODO list based on changes: |
I'm going to release 2.0 next week, moving into master branch position, and put the current master as a 1.x branch. I'll get to the list above except for "PhraseBuilder" which remains to be the reason to use 1.1. I'll push 2.0 to Maven as well. |
I think I'll close this as it's not going to happen (RE PhraseBuilder). The ConcatenateFilterFactory could be changed to use GraphTokenStreamFiniteStrings to accomplish the ultimate effect, and it wouldn't even be that hard AFAICT. |
I'd love to see the improvements made on the 1.2 branch ported to the MemPF branch (v2.0).
MemPF seems to work well in OpenSextant but it hasn't been as throughly evaluated. I suspect if Stanbol ports to MemPF, and if Rupert does his measurements as he's done before, it will be more clear through its tests, etc. how well the MemPF does.
The text was updated successfully, but these errors were encountered: