Разработчики мультимедийного пакета FFmpeg попросили компанию AMD внимательнее относится подготовке  патчей и не отправлять сгенерированные через AI изменения без проведения ручного рецензирования. Недовольство вызвал набор патчей с реализацией возможности использования  AMD HIP SDK (Heterogeneous-compute Interface for Portability) на платформе Windows для ускорения обработки видео на системах с GPU AMD...

Подробнее: https://www.opennet.ru/opennews/art.shtml?num=64707

• Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:12 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:17 , 30-Янв-26
    • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,expert, 13:18 , 30-Янв-26
      • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,КО, 13:26 , 30-Янв-26
        • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:34 , 30-Янв-26
          • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,КО, 13:37 , 30-Янв-26
          • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,nox., 13:39 , 30-Янв-26
            • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,User, 13:44 , 30-Янв-26
              • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,КО, 14:04 , 30-Янв-26
            • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 14:38 , 30-Янв-26
      • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:32 , 30-Янв-26
        • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,КО, 13:35 , 30-Янв-26
    • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:19 , 30-Янв-26
    • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,kravich, 13:22 , 30-Янв-26
      • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:54 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:18 , 30-Янв-26
• Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:18 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,kravich, 13:23 , 30-Янв-26
• Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:27 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,kravich, 13:36 , 30-Янв-26
    • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,КО, 13:39 , 30-Янв-26
      • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,kravich, 13:41 , 30-Янв-26
        • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,КО, 13:50 , 30-Янв-26
          • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,kravich, 13:53 , 30-Янв-26
        • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 14:16 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:39 , 30-Янв-26
    • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,kravich, 13:43 , 30-Янв-26
• Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:30 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:37 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,КО, 13:44 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,User, 13:46 , 30-Янв-26
• Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:31 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:34 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:54 , 30-Янв-26
• Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 13:36 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,nox., 13:45 , 30-Янв-26
    • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 14:06 , 30-Янв-26
• Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 14:14 , 30-Янв-26
• Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 14:22 , 30-Янв-26
  • Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 14:32 , 30-Янв-26
• Разработчики FFmpeg раскритиковали AMD за раздутые патчи,Аноним, 14:30 , 30-Янв-26

Вайбкодинг во всей красе, добро пожаловать в новый и прекрасный мир

> Автор патча сообщил, что осознанно добавил инструкцию по использованию pacman,
> написанную им 4 года назад
> Вайбкодинг

Какой вайбкодинг?
4 года назад еще никакого вайбкодинга не было.

> const int EIGHT = 8

А это все спецы по шаблонному мышлению виноваты. Мол магические константы в коде антипаттерн - выносите определение в именованную константу. Ну чел и вынес. Ну и дал самое логичное имя.

Недавно похожий случай был. Один рецензент покритиковал мой код. Ему не понравилось, что счетчики цикла имеют очень короткие и неинформативные, с его точки зрения, имена - i, j, k.

И как таких без вайбкодинга обходить?

Конечно, вместо i надо было написать COUNTEROFTHEFIRSTCYCLE. Круто, чо.

Не color, a colour!
В смысле CounterOfTheFirstCycle ну или COUNTER_OF_THE...
Остроконечники с тупоконечниками еще не договорились, приходите пожжа

не то  не другое - у вас стили наименования не подходящие к переменным, могут быть путаницы с константами и классами. :)

Надо римскими числами: i, ii, iii, iv, v, vi,  ...

Никаких магических переменных все ясно и понятно какие вопросы? Надо было const X = 8?

const int TRINITY=8;

иксперды во всей красе

А ты взял и поверил оправданиям гогнокодера?

> А ты взял и поверил оправданиям гогнокодера?

Я поверил пруфу, который привел разработчик
github.com/GPUOpen-LibrariesAndSDKs/AMF/wiki/Build-FFmpeg-with-AMF-Support/_history

А ты по какой-то причине называешь его гогнокодером.

И это компания, которая делала хорошие процы...

Ничего, ничего. Скоро уже ffmpeg-rs силу наберёт и заменят этих дидов наяривающих на ассемблер и недовольных патчами видите ли. Сами давно на грантах сидят и на развитие ffmpeg забили. Нет, чтобы спасибо сказать и в ножки покланяться за патч. Не хочу - хочу сапогом место указать на дверь. Ну и останутся тогда, как говорится, у дырявого корытца.

А потом к вам в ffmpeg-rs пришлют патч с константой EIGHT, и на опеннете появится новость "Разработчики FFmpeg-rs раскритиковали AMD за раздутые патчи"

К сожалению, полноценный ревью сгенерированного ллм кода лишает смысла использование ллм в целом из-за приближения затрат на ревью без ллм кода, созданного ллм, к затратам на написание кода и ревью без ллм. Поэтому стоит ожидать, что какое-то количество мертвого кода, добавление которого концептуально свойственно всем ai-генераторам, будет пролезать через тесты (особенно с учетом того, что тесты тоже пишут ОНИ).

В общем, ситуация та же, как с ревью кода джуна, когда ревьювер вынужден руками джуна полностью переписать изначальное решение

Ну он так джуна научит. Хотя и так научит писать правильные промпты. Правда, потом окажется, что написать правильный промпт сложнее, чем сразу на целевом яп.

Именно. С джуном эти упражнения имеют смысл, так как джун научится, а с заклинателями ~змей~ LLM-ок из-за отсутствия обратной связи все это бесполезно, и является просто изощренным и извращенным способом переложить работу с сабмиттера на ревьювера

Не - учит писать промпты.
Просто к промтам надо относиться как к ЯП 4 поколения.
Вот если за промптом потом надо подчищать в целевом ЯП - тогда да мартышкин труд.

Ок, возможно.
Но по прежнему сомнительно, что выразительности промптов достаточно для того, чтобы сделать все возможные правки, которые могут потребоваться
В общем, поживем-увидим

Для LLM уже придумали костыль - называется memory servers или MCP.

Слабые разработчики теперь называют любой непонятный код сгенерировапным ИИ и даже не пытаются разобраться. Они не в состоянии понять что const в с++ это переменная которая будет в особом сегменте памяти, а для сверхоптимизации и производительности такое бывает нужно.

Расказывая такие истории, нужно бы понимать, что сырая константа 8 вообще скорее всего будет константой в машинной инструкции инициализации регистра значением, что не требует отдельной транзакции памяти и потому еще быстрее

> разработчики, которые не видят отличий между wiki-страницей и сообщением в Git-коммите

Неужели и коммиты ИИ пишет? А EIGHT = 8 это распространенный среди разрабов кретинизм. Бывает ещё EMPTY = "" и всяческие попытки заменить нуль "пустой" при этом типизированной константой.

По-вашему, вместо константы M_PI следует везде в тексте написать 3.1415926536 и т.д.?

Можа и кретинизм, а на RISC архитектуре это один раз доступная константа, а не каждый раз перепрыгивать константу в коде. Для CISC, что в лоб, что по лбу.

> Неужели и коммиты ИИ пишет?

Ну вообще да - в выстроенном процессе разработки с саммаризацией он как раз таки хорошо справляется.

Пф, писаки FFmpeg открыли рот чтобы кого-то критиковать?
Чуваки, вы ж не единственные разработчики мультимедийных пакетов.
Или принимайте патчи как удобно настоящим разработчикам, или останетесь вообще без патчей.

Вот вот! И будут писать про свой ассемблер раз в 5 лет как они какой-то никому не нужный древний кодек ЯКОБЫ ускорили в 50 миллиардов раз написав код ассемблер руками.Тем временем более сговорчевые гигачеды-модерн-стайл-разрабы с нормальными софт-скиллами (а не в стиле Линуса-посылателя) будут получать всё больше ПРов и наслаждаться патчами, донатами и заслуженным (!) признанием, пока диды ffmpeg будут яростно скриметь вставными челюстями грозя, что вот ещё 20 патчей ручного ассемблера и вот они как вернуться, да как всех сместят!

>вы ж не единственные разработчики мультимедийных пакетов

А другие прям рады всё это разгрeбать ?

В военное время значение EIGHT может достигать 9, а то 10. Надо понимать.

Зря шутите. Недавно разбирал задания по физике для продвинутых детей. В комментариях было указано - для расчетов ускорение свободного падения на Земле g взять равным 10.

А что не так?
Если тебе нужно оценить сверху макс скорость которую разовьет белка падая после скольжения по абсолютно ровному столу.....

Если у белки есть шанс, то можно пересчитать с точным значением))

Считал у ффспег один разраб всего. Если их много,то вполне потянут и будет ффмпег размером с ядро благодаря амуде.

> В частности, в коде имелась константа с именем 8 и значением 8 (const int EIGHT = 8),

Справедливости ради: в моем коде тоже есть подобное. Потому что имя идентификатора не может начинаться с цифры, поэтому 32-bit превращается в THIRTY_TWO_BIT.

А ты почему пишешь низкокачественный код? Ты видимо не уважаешь сообщество. Ты какой-то не умный человек, да?!

Коммерческие компании всегда разабатывали и разрабатывают низкокачественный код.