-
Notifications
You must be signed in to change notification settings - Fork 302
Provide a long-term policy for LLM usage #2260
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?
Conversation
|
I wouldn't make this official yet, it's not a fine line and my statement in #2258 was non-conclusive. While I personally have opinions about proper responsible use of artificial intelligence, I'd wait on @GarboMuffin's opinion on really anything regarding official contributing guidelines. |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Brackets-Coder
left a comment
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.
Tricky issue...
|
@Brackets-Coder I started a review by accident. Please ignore it. |
|
@Brackets-Coder @GarboMuffin What do you think of the latest revision? |
Brackets-Coder
left a comment
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.
This is still a tricky issue, I'm going to need @GarboMuffin opinions.
|
#2286 appears to have been assisted by an LLM. This is getting worse every day that my policy improvements are delayed. |
|
@SharkPool-SP @CubesterYT I want some more perspectives on this. What do you all think? |
|
i am not presently concerned about the lack of an LLM policy in that document |
|
Closed because comment above |
|
I am reopening this because #2327 gives us authority to approve CONTRIBUTING.MD changes ourselves. |
Predates new review procedures.
|
#1828 and #2338 are also disclosed as having used LLMs. @Brackets-Coder @CubesterYT What do you think about approving this now that the new review process is in place? |
|
I am not convinced that we need an LLM policy. People seem to be largely honest about it. Having a no LLM policy will still result in LLM spam they just won't be honest about it. Have you found any significant functional issues in those extensions? |
|
I'm not deeply concerned about an LLM policy, if an extension is poorly written we won't approve it anyway |
I'm mostly concerned with having some procedure to quickly reject LLM spam without having to treat it with the respect of someone's actual hard work. Currently, there is not a well-defined procedure for handling such extensions, and the PRs will often stay open for several months before getting closed. As for functional issues, take a look at #2338. Their proposal incorrectly claims that "all strings...include German, Spanish, French, etc., translations within the file". If you review their source code, however, you will notice a complete lack of such functionality. They also added an apparently-hallucinated This is the sort of code quality that will slowly destroy any large project, TurboWarp included. |
In 2023, @GarboMuffin made a temporary LLM policy but then removed it a year later. #2258 brought up the possibility that LLM-written code may be able to pass code review with the current state of things, and this would be a bleak reality for our userbase.
This pull request makes a more permanent set of guidelines that were written in the spirit of GarboMuffin's original ideas, but expanded to account for the modern usage of LLMs and the increasing number of image generators.