odinson
odinson copied to clipboard
Integrate BioNLPProcessor with Odinson
This PR adds the BioNLPProcessor as one of the processor choices used for annotation. See https://github.com/clulab/reach/issues/802.
WARNING: review for changing dependencies (this only works if Reach is available as a dependency) and changing default behavior (for illustrative purposes I changed application.conf to choose the BioNLPProcessor). Further changes may be needed to mitigate any unintended consequences of these.
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.
Thanks, @bgyori. I suspect that we might not want to incorporate this into the main branch, but it would be very nice to have the changes around for documentation or to serve as an example. Maybe we here can incorporate them into a branch for safe keeping. What does @myedibleenso who knows both reach and odinson think?