by Alifatisk
1 subcomments
- The dialog was something to read. Doesn’t sound like ai but rather something a developer has practiced for years and no one has challenged the dev on this, until now.
Also, there is discussions on what a commit message should contain, apparently the patch had user guidance in the commit message. Literally on how to install dependencies such as gcc using pacman, in the commit message.
https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21595
- const int EIGHT = 8 lol
I really doubt any AI (even some small local models) would actually generate something like this :)
by high_na_euv
3 subcomments
- I dont see problem with user manual in commit as long as reasoning for the commit is clearly written
by estimator7292
0 subcomment
- The xitter post consists of only a screenshot, a link, and a drive-by comment.
Here's the actual thing we're talking about https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21595#issuecomme...
- AI code, junior dev or distracted person, either way there is no defense for AMD. As a billion dollar company you would at least have a simple QA before pushing slop into open source projects
FFmpeg maintainers have to keep pointing out incorrect parts. And conclude the entire is redundant
The PR interaction are clearly AI filler, at least on the second one
At this point open source projects should charge big companies to review their PRs