odinson icon indicating copy to clipboard operation
odinson copied to clipboard

Integrate BioNLPProcessor with Odinson

Open bgyori opened this issue 1 year ago • 2 comments

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.

bgyori avatar Oct 18 '23 02:10 bgyori

CLA assistant check
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.

CLAassistant avatar Oct 18 '23 02:10 CLAassistant

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?

kwalcock avatar Oct 20 '23 16:10 kwalcock