The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



"Разработчики FFmpeg раскритиковали AMD за раздутые патчи"
Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Изначальное сообщение [ Отслеживать ]

"Разработчики FFmpeg раскритиковали AMD за раздутые патчи"  +/
Сообщение от opennews (??), 30-Янв-26, 13:12 
Разработчики мультимедийного пакета FFmpeg попросили компанию AMD внимательнее относится подготовке  патчей и не отправлять сгенерированные через AI изменения без проведения ручного рецензирования. Недовольство вызвал набор патчей с реализацией возможности использования  AMD HIP SDK (Heterogeneous-compute Interface for Portability) на платформе Windows для ускорения обработки видео на системах с GPU AMD...

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

Ответить | Правка | Cообщить модератору

Оглавление

Сообщения [Сортировка по ответам | RSS]

1. Сообщение от Аноним (1), 30-Янв-26, 13:12   +13 +/
Вайбкодинг во всей красе, добро пожаловать в новый и прекрасный мир
Ответить | Правка | Наверх | Cообщить модератору
Ответы: #2, #5

2. Сообщение от Аноним (-), 30-Янв-26, 13:17   +1 +/
> Автор патча сообщил, что осознанно добавил инструкцию по использованию pacman,
> написанную им 4 года назад
> Вайбкодинг

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

Ответить | Правка | Наверх | Cообщить модератору
Родитель: #1 Ответы: #4, #6, #8

3. Сообщение от Аноним (3), 30-Янв-26, 13:18    Скрыто ботом-модератором–1 +/
Ответить | Правка | Наверх | Cообщить модератору
Ответы: #9

4. Сообщение от expert (??), 30-Янв-26, 13:18   +11 +/
> const int EIGHT = 8
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #2 Ответы: #10, #14

5. Сообщение от Аноним (5), 30-Янв-26, 13:18   +4 +/
И это компания, которая делала хорошие процы...
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #1 Ответы: #45

6. Сообщение от Аноним (6), 30-Янв-26, 13:19   +2 +/
иксперды во всей красе
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #2

8. Сообщение от kravich (ok), 30-Янв-26, 13:22   +2 +/
А ты взял и поверил оправданиям гогнокодера?
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #2 Ответы: #34

9. Сообщение от kravich (ok), 30-Янв-26, 13:23   +1 +/
А потом к вам в ffmpeg-rs пришлют патч с константой EIGHT, и на опеннете появится новость "Разработчики FFmpeg-rs раскритиковали AMD за раздутые патчи"
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #3

10. Сообщение от КО (?), 30-Янв-26, 13:26   +/
А это все спецы по шаблонному мышлению виноваты. Мол магические константы в коде антипаттерн - выносите определение в именованную константу. Ну чел и вынес. Ну и дал самое логичное имя.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #4 Ответы: #15

11. Сообщение от Аноним (11), 30-Янв-26, 13:27   +1 +/
К сожалению, полноценный ревью сгенерированного ллм кода лишает смысла использование ллм в целом из-за приближения затрат на ревью без ллм кода, созданного ллм, к затратам на написание кода и ревью без ллм. Поэтому стоит ожидать, что какое-то количество мертвого кода, добавление которого концептуально свойственно всем ai-генераторам, будет пролезать через тесты (особенно с учетом того, что тесты тоже пишут ОНИ).
Ответить | Правка | Наверх | Cообщить модератору
Ответы: #19, #22

12. Сообщение от Аноним (12), 30-Янв-26, 13:30   –3 +/
> разработчики, которые не видят отличий между wiki-страницей и сообщением в Git-коммите

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

Ответить | Правка | Наверх | Cообщить модератору
Ответы: #20, #27, #30, #46

13. Сообщение от Аноним (-), 30-Янв-26, 13:31   –5 +/
Пф, писаки FFmpeg открыли рот чтобы кого-то критиковать?
Чуваки, вы ж не единственные разработчики мультимедийных пакетов.
Или принимайте патчи как удобно настоящим разработчикам, или останетесь вообще без патчей.
Ответить | Правка | Наверх | Cообщить модератору
Ответы: #16, #33

14. Сообщение от Аноним (14), 30-Янв-26, 13:32   +/
Никаких магических переменных все ясно и понятно какие вопросы? Надо было const X = 8?
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #4 Ответы: #17

15. Сообщение от Аноним (15), 30-Янв-26, 13:34   +6 +/
Недавно похожий случай был. Один рецензент покритиковал мой код. Ему не понравилось, что счетчики цикла имеют очень короткие и неинформативные, с его точки зрения, имена - i, j, k.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #10 Ответы: #21, #24

16. Сообщение от Аноним (3), 30-Янв-26, 13:34   –1 +/
Вот вот! И будут писать про свой ассемблер раз в 5 лет как они какой-то никому не нужный древний кодек ЯКОБЫ ускорили в 50 миллиардов раз написав код ассемблер руками.Тем временем более сговорчевые гигачеды-модерн-стайл-разрабы с нормальными софт-скиллами (а не в стиле Линуса-посылателя) будут получать всё больше ПРов и наслаждаться патчами, донатами и заслуженным (!) признанием, пока диды ffmpeg будут яростно скриметь вставными челюстями грозя, что вот ещё 20 патчей ручного ассемблера и вот они как вернуться, да как всех сместят!
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #13

17. Сообщение от КО (?), 30-Янв-26, 13:35   +2 +/
const int TRINITY=8;
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #14

18. Сообщение от Аноним (18), 30-Янв-26, 13:36   +7 +/
В военное время значение EIGHT может достигать 9, а то 10. Надо понимать.
Ответить | Правка | Наверх | Cообщить модератору
Ответы: #29

19. Сообщение от kravich (ok), 30-Янв-26, 13:36   +1 +/
В общем, ситуация та же, как с ревью кода джуна, когда ревьювер вынужден руками джуна полностью переписать изначальное решение
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #11 Ответы: #23

20. Сообщение от Аноним (15), 30-Янв-26, 13:37   –3 +/
По-вашему, вместо константы M_PI следует везде в тексте написать 3.1415926536 и т.д.?
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #12

21. Сообщение от КО (?), 30-Янв-26, 13:37   +/
И как таких без вайбкодинга обходить?
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #15

22. Сообщение от Аноним (14), 30-Янв-26, 13:39   –4 +/
Слабые разработчики теперь называют любой непонятный код сгенерировапным ИИ и даже не пытаются разобраться. Они не в состоянии понять что const в с++ это переменная которая будет в особом сегменте памяти, а для сверхоптимизации и производительности такое бывает нужно.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #11 Ответы: #26

23. Сообщение от КО (?), 30-Янв-26, 13:39   +1 +/
Ну он так джуна научит. Хотя и так научит писать правильные промпты. Правда, потом окажется, что написать правильный промпт сложнее, чем сразу на целевом яп.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #19 Ответы: #25

24. Сообщение от nox. (?), 30-Янв-26, 13:39   +3 +/
Конечно, вместо i надо было написать COUNTEROFTHEFIRSTCYCLE. Круто, чо.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #15 Ответы: #28, #42

25. Сообщение от kravich (ok), 30-Янв-26, 13:41   +1 +/
Именно. С джуном эти упражнения имеют смысл, так как джун научится, а с заклинателями ~змей~ LLM-ок из-за отсутствия обратной связи все это бесполезно, и является просто изощренным и извращенным способом переложить работу с сабмиттера на ревьювера
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #23 Ответы: #31, #38, #49

26. Сообщение от kravich (ok), 30-Янв-26, 13:43   +3 +/
Расказывая такие истории, нужно бы понимать, что сырая константа 8 вообще скорее всего будет константой в машинной инструкции инициализации регистра значением, что не требует отдельной транзакции памяти и потому еще быстрее
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #22

27. Сообщение от КО (?), 30-Янв-26, 13:44   +/
Можа и кретинизм, а на RISC архитектуре это один раз доступная константа, а не каждый раз перепрыгивать константу в коде. Для CISC, что в лоб, что по лбу.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #12

28. Сообщение от User (??), 30-Янв-26, 13:44   +1 +/
Не color, a colour!
В смысле CounterOfTheFirstCycle ну или COUNTER_OF_THE...
Остроконечники с тупоконечниками еще не договорились, приходите пожжа
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #24 Ответы: #35

29. Сообщение от nox. (?), 30-Янв-26, 13:45   +/
Зря шутите. Недавно разбирал задания по физике для продвинутых детей. В комментариях было указано - для расчетов ускорение свободного падения на Земле g взять равным 10.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #18 Ответы: #36

30. Сообщение от User (??), 30-Янв-26, 13:46   +/
> Неужели и коммиты ИИ пишет?

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

Ответить | Правка | Наверх | Cообщить модератору
Родитель: #12

31. Сообщение от КО (?), 30-Янв-26, 13:50   +/
Не - учит писать промпты.
Просто к промтам надо относиться как к ЯП 4 поколения.
Вот если за промптом потом надо подчищать в целевом ЯП - тогда да мартышкин труд.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #25 Ответы: #32

32. Сообщение от kravich (ok), 30-Янв-26, 13:53   +/
Ок, возможно.
Но по прежнему сомнительно, что выразительности промптов достаточно для того, чтобы сделать все возможные правки, которые могут потребоваться
В общем, поживем-увидим
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #31

33. Сообщение от Аноним (33), 30-Янв-26, 13:54   +4 +/
>вы ж не единственные разработчики мультимедийных пакетов

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

Ответить | Правка | Наверх | Cообщить модератору
Родитель: #13

34. Сообщение от Аноним (34), 30-Янв-26, 13:54    Скрыто ботом-модератором–1 +/
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #8

35. Сообщение от КО (?), 30-Янв-26, 14:04   +/
не то  не другое - у вас стили наименования не подходящие к переменным, могут быть путаницы с константами и классами. :)
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #28

36. Сообщение от Аноним (-), 30-Янв-26, 14:06   +3 +/
А что не так?
Если тебе нужно оценить сверху макс скорость которую разовьет белка падая после скольжения по абсолютно ровному столу.....

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

Ответить | Правка | Наверх | Cообщить модератору
Родитель: #29

37. Сообщение от Аноним (37), 30-Янв-26, 14:14   +/
Считал у ффспег один разраб всего. Если их много,то вполне потянут и будет ффмпег размером с ядро благодаря амуде.
Ответить | Правка | Наверх | Cообщить модератору

38. Сообщение от Аноним (38), 30-Янв-26, 14:16   +/
Для LLM уже придумали костыль - называется memory servers или MCP.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #25

39. Сообщение от Аноним (39), 30-Янв-26, 14:22   +/
> В частности, в коде имелась константа с именем 8 и значением 8 (const int EIGHT = 8),

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

Ответить | Правка | Наверх | Cообщить модератору
Ответы: #41

40. Сообщение от Аноним (-), 30-Янв-26, 14:30    Скрыто ботом-модератором+2 +/
Ответить | Правка | Наверх | Cообщить модератору

41. Сообщение от Аноним (-), 30-Янв-26, 14:32    Скрыто ботом-модератором+/
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #39

42. Сообщение от Аноним (42), 30-Янв-26, 14:38   +1 +/
Надо римскими числами: i, ii, iii, iv, v, vi,  ...
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #24

43. Сообщение от Аноним (-), 30-Янв-26, 14:53   –1 +/
АМД старается как может а эти коты неблагодарные только жалуются!
а может они чемоданы от Невидии получили...
короче все переходим на открытыйLibav
Ответить | Правка | Наверх | Cообщить модератору

44. Сообщение от pofigist (?), 30-Янв-26, 15:13   +/
Результат неприятия LLM в текущих проектах, будут форум этих проектов, которые и получат все гранты.
Ибо без патчей от корпораций типа AMD - они никому не нужны, а корпы - не будут писать эти патчи ручками ибо LLM - дешевле.
Ответить | Правка | Наверх | Cообщить модератору
Ответы: #48

45. Сообщение от нах. (?), 30-Янв-26, 15:26   +/
тоже нет.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #5

46. Сообщение от нах. (?), 30-Янв-26, 15:27   +/
> Неужели и коммиты ИИ пишет?

с разморозочкой. Тебе у нас - не понравится!

Ответить | Правка | Наверх | Cообщить модератору
Родитель: #12

47. Сообщение от Анонимомус (?), 30-Янв-26, 15:29   +/
> В ответ представитель FFmpeg извинился за непонимание и сказал, что даже не представлял, что существуют разработчики, которые не видят отличий между wiki-страницей и сообщением в Git-коммите.

Вроде и извинился, а вроде и послал, по делу

Ответить | Правка | Наверх | Cообщить модератору

48. Сообщение от похнапоха (?), 30-Янв-26, 15:30    Скрыто ботом-модератором+/
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #44

49. Сообщение от kusb (?), 30-Янв-26, 15:30   +/
>змей

Шогготов?

Ответить | Правка | Наверх | Cообщить модератору
Родитель: #25


Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2026 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру