Adding to my previous reply, it would be great to also get OPENNLP-1384 into the minor release. I will try to get a PR for it up this weekend.
https://issues.apache.org/jira/browse/OPENNLP-1384 Thanks, Jeff On Fri, Oct 13, 2023 at 8:50 AM Jeff Zemerick <jzemer...@apache.org> wrote: > I think it would be good to do a minor release to support Apache Solr as > long as it doesn't put too much burden on any of us at the moment. > > Thanks, > Jeff > > On Fri, Oct 13, 2023 at 1:55 AM Atita Arora <atitaar...@gmail.com> wrote: > >> Sounds fair and reasonable to me. >> >> Thanks, >> Atita >> >> On Thu, 12 Oct 2023, 21:50 Richard Zowalla, <r...@apache.org> wrote: >> >> > I am fine with the suggested approach. >> > >> > Maybe, we should do a minor release after this is merged in order to >> > unblock the SOLR folks? >> > >> > Gruß >> > Richard >> > >> > Am Donnerstag, dem 12.10.2023 um 12:41 -0400 schrieb Jeff Zemerick: >> > > Hi all, >> > > >> > > I created OPENNLP-1515 to change the ONNX Runtime dependency from >> > > onnxruntime-gpu to onnxruntime. This change will remove GPU support >> > > and >> > > cause OpenNLP to always use CPU for inference. The reason for this >> > > change >> > > is the onnxruntime dependency supports Linux, Windows, and Mac x64, >> > > and the >> > > onnxruntime-gpu dependency only supports Linux and Windows. ( >> > > https://onnxruntime.ai/docs/get-started/with-java.html) I think >> > > OpenNLP >> > > should support the most operating systems out of the box instead of >> > > favoring GPU. Please take a look at the pull request: >> > > https://github.com/apache/opennlp/pull/551 >> > > >> > > This change is partially proposed to support OpenNLP's ONNX >> > > integration in >> > > Apache Solr: https://github.com/apache/solr/pull/1999 >> > > >> > > I think GPU support in OpenNLP should be easily accessible to users, >> > > so I >> > > wrote OPENNLP-1516 to capture that with a link to one possible >> > > method. If >> > > the above PR is merged, a user can still enable GPU in OpenNLP by >> > > manually >> > > replacing the onnxruntime.jar with onnxruntime-gpu.jar on their >> > > classpath >> > > until OPENNLP-1516 is resolved. >> > > >> > > All comments/suggestions are welcome! >> > > >> > > Thanks, >> > > Jeff >> > >> > >> >