-
Notifications
You must be signed in to change notification settings - Fork 28
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
Reconfiguration of Annotators #212
base: master
Are you sure you want to change the base?
Conversation
…ne. Now a communication between Knowledge and RS needs to be implemented via PrologCPPQueries and some additional rules to check if a annotator can be reconfigured to meet the requested requirements. In order to achieve this final communication, an additonal reconfiguration service needs to be implemented.
…ix name resolving.
@Sanic you'll look through this ? if you need help let me know. It's been a while since I looked at the code. I would def not recommend merging without having the code coverage increase. Some of these changes affect basic pipeline planning capabilities and we should make sure that they still work |
Hi @bbferka But you also raise a good point to test also the basic planning capabilities. @evankapi Could you please keep that in mind? |
This extension includes reconfigurable annotators and updates to the query system to handle different inputs / outputs of annotator setups. Those setups are also considered during the pipeline planning via the query service.