mawiesne commented on code in PR #1462:
URL: https://github.com/apache/opennlp/pull/1462#discussion_r4280971419
##########
opennlp-docs/src/docbkx/machine-learning.xml:
##########
@@ -183,5 +183,26 @@ try (TextEmbedder embedder =
providers.select(spec).create(spec)) {
has no provider until another one is installed.
Existing <code>SentenceVectorsDL</code>
constructors remain available.
</para>
+ <para>
+ To compute the vectors on a GPU, depend on
<code>opennlp-dl-gpu</code> in place of
+ <code>opennlp-dl</code> and pass
<code>InferenceOptions</code> to
+ <code>SentenceVectorsDL</code>.
<code>opennlp-dl-gpu</code> supports linux-x64 and
Review Comment:
Does this line still hold? ("win + linux" only)
What about macos with nvidia? Can you quickly check, and potentially add a
note on MacOS users whether those are able to use it or not. Please add a
specific (in-/exclusion) comment, or change the sentence to cover all three as
supported.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
