-
Notifications
You must be signed in to change notification settings - Fork 2.1k
CONTRIBUTION.md: some sentences about LLM #22004
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Changes from all commits
924a3c0
95b8d7c
728806d
181e8d8
62887cd
9135733
55a6f0f
fcc133d
22c4474
998f963
7c11752
1be3d32
7943003
10291a0
24a4ce2
6f6e8a0
5b3bbfc
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -208,6 +208,26 @@ into the source repository. | |||||
| [archived-pull-requests]: https://github.com/RIOT-OS/RIOT/pulls?q=is:pr+label:"State:+archived" | ||||||
| [uncrustify]: http://uncrustify.sourceforge.net | ||||||
|
|
||||||
| ## Plagiarism and AI | ||||||
|
|
||||||
| RIOT itself applies the LGPL license, see [LICENSE.md], to most of its code exclusively, | ||||||
| authors of PRs are assumed to do so as well (not necessary exclusive). | ||||||
|
Comment on lines
+213
to
+214
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'd argue that these are two different sentences
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
I'd honestly just remove this, the same statement is covered in the next sentence. |
||||||
| When a PR is published it must also respect all authorship-rights, copyrights and | ||||||
AnnsAnns marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
| licenses of code it uses. If code is copied the original author usually must be named and | ||||||
| the original license must be kept, unless the original license states differently. | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think we should tone it down a bit for the common pattern of copying an existing board / driver /example and starting from there. We aren't interested in the pedigree of the boilerplate, if it ends up being a complete rewrite the author of the skeleton is just a distraction. I've seen this a couple of times in the past and always suggested to drop the old |
||||||
| [Generative AI] (e.g., Large language models (LLM)) are known to sometimes reproduce | ||||||
| large amounts of content (code), that was fed into their database as such it might | ||||||
| produce a copy and the previous sentence must be applied. | ||||||
| When reworking a PR and copying its code to a new one, the author of that original reproduced | ||||||
| code still is the original author and their rights to the code must be respected | ||||||
| such as naming them and keeping the license. | ||||||
|
|
||||||
| If a PR was written using tooling that is able to reproduce code from a knowlege-base, e.g., | ||||||
| large language model assisted tools like Copilot, Devin AI or Cursor or LLMs like GPT and Ollama, | ||||||
| the PR should be marked as such and describe to what extent the tool was used | ||||||
| (producing whole files, functions, code blocks, documentation, code/spell checking). | ||||||
| [Generative AI]: https://en.wikipedia.org/wiki/Generative_artificial_intelligence | ||||||
|
|
||||||
| ## Writing Documentation | ||||||
| [writing-documentation]: #writing-documentation | ||||||
|
|
||||||
|
|
||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.