From 272d385a0ebbef953759536e97b7f0171f2a0828 Mon Sep 17 00:00:00 2001 From: nickbianco Date: Mon, 5 Oct 2026 13:18:49 -0500 Subject: [PATCH 1/3] Add AI policy to contributors guide and update PR template --- .github/PULL_REQUEST_TEMPLATE.md | 5 +++++ CONTRIBUTING.md | 15 ++++++++++++++- 2 files changed, 19 insertions(+), 1 deletion(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 81f982466e..a69cfae16a 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -6,7 +6,12 @@ Fixes issue # ### Looking for feedback on... +### AI/LLM usage + + ### CHANGELOG.md (choose one) - no need to update because... - updated. + +Please review the [CONTRIBUTING.md](../CONTRIBUTING.md) file for detailed contributing guidelines. \ No newline at end of file diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index ae79b3e652..ae35547858 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -11,6 +11,7 @@ Contents: - [Ways to Contribute](#ways-to-contribute) - [Making a Pull Request](#making-a-pull-request-pr) +- [AI/LLM Usage Policy](#aillm-usage-policy) - [Writing tests with Catch2](#writing-tests-with-catch2) - [Running Moco tests](#running-moco-tests) - [Checking for Memory Leaks with LibASAN](#checking-for-memory-leaks-with-libasan) @@ -35,7 +36,6 @@ There are lots of ways to contribute to the OpenSim project, and people with wid Seth A, Hicks JL, Uchida TK, Habib A, Dembia CL, Dunne JJ, Ong CF, DeMers MS, Rajagopal A, Millard M, Hamner SR, Arnold EM, Yong JR, Lakshmikanth SK, Sherman MA, Ku JP, Delp SL (2018) OpenSim: Simulating musculoskeletal dynamics and neuromuscular control to study human and animal movement. *PLoS Computational Biology* 14(7):e1006223. - Making a Pull Request (PR) -------------------------- Please don't surprise us with a big out-of-the-blue pull request (PR). Preferably, target an existing Issue and post a comment to that effect in the Issue. That way, you can engage in discussion with others to coordinate your work with theirs and avoid duplication, see what people think of the approach you have planned, and so on. If there is not yet a relevant Issue in place, a great way to start is by creating one, and then engaging in an Issue conversation. @@ -68,6 +68,19 @@ A few additional practices will help streamline the code review process. Please It is important that reviewers also review the effect that your PR has on the doxygen documentation. To facilitate this, we automatically upload the doxygen documentation for each PR to [myosin.sourceforge.net](http://myosin.sourceforge.net); you can view the documentation for a specific PR at `myosin.sourceforge.net/`. + +AI/LLM Usage Policy +------------------- +Contributors to OpenSim are welcome to use Artificial Intelligence (AI) or Large Language Model (LLM) coding tools to create pull requests. However, all pull requests must be created with a human in the loop. The contributor is always the author and is fully accountable for their contributions. Contributors should be confident that their submission is high quality and worth the (limited) time of the maintainers. + +- You must disclose in the pull request description that you used an AI/LLM tool or model and how it was used. +- You must review all AI/LLM-generated code, comments, or documentation before requesting a review. +- You must not attribute an AI/LLM model authorship to any commit in a pull request (e.g., "Co-Authored by Claude"). +- You must answer all maintainer questions and pull request review comments yourself, without using an LLM or AI chatbot. + +Maintainers may close any pull requests that do not follow the guidelines above. + + Writing tests with Catch2 ------------------------- Tests should be written using the [Catch2 testing framework](https://github.com/catchorg/Catch2). From 980c39898911ddebaec274958dd510899707eb89 Mon Sep 17 00:00:00 2001 From: nickbianco Date: Wed, 7 Oct 2026 09:18:57 -0500 Subject: [PATCH 2/3] Address review comments --- .github/PULL_REQUEST_TEMPLATE.md | 4 ++-- CONTRIBUTING.md | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index a69cfae16a..0508b23e65 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -6,8 +6,8 @@ Fixes issue # ### Looking for feedback on... -### AI/LLM usage - +### AI/LLM use + ### CHANGELOG.md (choose one) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index ae35547858..b937ec7c5c 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -71,7 +71,7 @@ It is important that reviewers also review the effect that your PR has on the do AI/LLM Usage Policy ------------------- -Contributors to OpenSim are welcome to use Artificial Intelligence (AI) or Large Language Model (LLM) coding tools to create pull requests. However, all pull requests must be created with a human in the loop. The contributor is always the author and is fully accountable for their contributions. Contributors should be confident that their submission is high quality and worth the (limited) time of the maintainers. +Contributors to OpenSim are welcome to use Artificial Intelligence (AI) or Large Language Model (LLM) coding tools to create pull requests. However, all pull requests must be created with a human in the loop. The contributor is always the author and is fully accountable for their contributions. Contributors must be confident that their submission is high quality and worth the (limited) time of the maintainers. - You must disclose in the pull request description that you used an AI/LLM tool or model and how it was used. - You must review all AI/LLM-generated code, comments, or documentation before requesting a review. From af125a9944df6fcb39fd389f4fa602cb8603ffb7 Mon Sep 17 00:00:00 2001 From: nickbianco Date: Wed, 7 Oct 2026 09:45:15 -0500 Subject: [PATCH 3/3] Add missed change in CONTRIBUTING.md [skip ci] --- CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index b937ec7c5c..ca745a3f5a 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -74,7 +74,7 @@ AI/LLM Usage Policy Contributors to OpenSim are welcome to use Artificial Intelligence (AI) or Large Language Model (LLM) coding tools to create pull requests. However, all pull requests must be created with a human in the loop. The contributor is always the author and is fully accountable for their contributions. Contributors must be confident that their submission is high quality and worth the (limited) time of the maintainers. - You must disclose in the pull request description that you used an AI/LLM tool or model and how it was used. -- You must review all AI/LLM-generated code, comments, or documentation before requesting a review. +- You must review all AI/LLM-generated code, comments, and documentation before requesting a review. - You must not attribute an AI/LLM model authorship to any commit in a pull request (e.g., "Co-Authored by Claude"). - You must answer all maintainer questions and pull request review comments yourself, without using an LLM or AI chatbot.