krickert commented on PR #1470: URL: https://github.com/apache/opennlp/pull/1470#issuecomment-6111900976
> Hi @krickert - thanks, good to have this. One point should be aligned before merging: the AI disclosure rules need to follow the ASF guidance. > > * The [ASF generative tooling guidance](https://www.apache.org/legal/generative-tooling.html) recommends a `Generated-by: <Name of Tool> <Version>` token in the commit message, and the [RAI commit-message practice](https://rai.apache.org/commit-messages.html) asks to name the actual tool and model rather than a generic label. The current "Do not add model identifiers" and the `Assisted-by:` trailer contradict both. Suggested wording: > > Disclose AI help in the commit message with a `Generated-by: <tool> <version>` trailer, naming the actual tool and model version, as recommended by the ASF generative tooling guidance. > * Please reference the ASF page as the primary policy; the Lucene AI policy can stay as an additional reference. > * Minor: "disucssion" → "discussion"; "enforce the user to post to discussions on GITHUB" → "ask on GitHub Discussions or [[email protected]](mailto:[email protected])"; "Standard JDK character and space tools are usually rewritten in OpenNLP" duplicates the following two bullets and can be removed. > > WDYT? I agree to show the tool, the version of the model and version of the harness. We agreed to follow the lucene standard too and had some standards in the email mentioned- totally agree. For the most part (pardon my french) "don't be a jerk" is the overall arching theme - but since it's an LLM, that's pretty subjective. I think all of the suggestions are spot-on - it's such an important piece that I expect a couple rounds of discussion on this one. I'm more of an AB test sorta guy, given the uncertainty of LLMs, I think whatever our first version is we'll find tweaks we need to do all the time. A year from now let's hope the code produced never looks mean to us humans. When I first used AI, I used to say "it must compile before I look at it" or "look at the source code instead of stack overflow" - that is no longer needed. -- 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]
