> “The primary purpose of the DATA statement is to give names to constants; instead of referring to pi as 3.141592653589793 at every appearance, the variable PI can be given that value with a DATA statement and used instead of the longer form of the constant. This also simplifies modifying the program, should the value of pi change.”
> — Early FORTRAN manual for Xerox Computers, attributed to David H. Owens.
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.
I get maintainers have their own issues to deal with, and respect that they are trying to keep the project clean. At work I have had many times where I spent more of my day reviewing MRs than actually writing code, and sometimes my cold blunt replies can unintentionally rub people the wrong way.
Still, I feel like they were pretty rude to this guy for no real reason. I don't think I'd want to work with them.
It's not about the commit itself but that due to it they discovered vibe coding by AMD. They object to the sending commits that have not been checked by a human dev.
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
9 comments
[ 2.5 ms ] story [ 35.8 ms ] threadI really doubt any AI (even some small local models) would actually generate something like this :)
> “The primary purpose of the DATA statement is to give names to constants; instead of referring to pi as 3.141592653589793 at every appearance, the variable PI can be given that value with a DATA statement and used instead of the longer form of the constant. This also simplifies modifying the program, should the value of pi change.”
> — Early FORTRAN manual for Xerox Computers, attributed to David H. Owens.
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
Still, I feel like they were pretty rude to this guy for no real reason. I don't think I'd want to work with them.
Here's the actual thing we're talking about https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21595#issuecomme...
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