Skip to content

UPSTREAM PR #18388: (proposal) contributing: tighten AI usage policy#711

Open
loci-dev wants to merge 1 commit intomainfrom
upstream-PR18388-branch_ggml-org-xsn/contrib_tighter_ai_policy
Open

UPSTREAM PR #18388: (proposal) contributing: tighten AI usage policy#711
loci-dev wants to merge 1 commit intomainfrom
upstream-PR18388-branch_ggml-org-xsn/contrib_tighter_ai_policy

Conversation

@loci-dev
Copy link
Copy Markdown

Mirrored from ggml-org/llama.cpp#18388

In my personal case, nitpicking AI-generated PRs took me quite a lot of time, which lead to net-0 or even negative productivity (even with the help of AI-assisted reviewing)

So I make this proposition to see if other maintainers agree. This is inspired by p5.js's policy.

Contributor often circumvent by saying that their code has gone though significant manual changes. But in practice, they don't change the structure of the code that much. Some even hide the fact that they don't understand the code by forwarding my questions/my reviews directly to AI. That's why I'm proposing now limit the usage of AI to be assisting-only, not for generating the first version of the code, then manually fix it.


This PR is written by me, proofreading by AI.

@loci-review
Copy link
Copy Markdown

loci-review bot commented Dec 26, 2025

Explore the complete analysis inside the Version Insights

Perfect! I've generated the summary report for your project. Here's what the analysis shows:

Performance Summary Report

Project Details:

Version Comparison:

  • Base Version: 442467b1-de25-42b2-95be-b99339dc7a41
  • Target Version: 823c7036-29ef-4c60-8892-3250bf0c3722

Summary

The performance analysis shows no significant performance regressions or improvements between the two versions.

Key Findings:

  • ✅ No modified functions with response time changes greater than 2%
  • ✅ No modified functions with throughput time changes greater than 2%

Conclusion: Pull Request #711 has a neutral performance impact and appears safe to merge from a performance perspective, as it maintains performance stability without introducing measurable regressions.

@loci-dev loci-dev force-pushed the main branch 27 times, most recently from 559165d to ae8ce6f Compare December 30, 2025 12:15
@loci-dev loci-dev force-pushed the main branch 30 times, most recently from 7b2e535 to bfd3c27 Compare January 5, 2026 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants