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   +23 +/
Вайбкодинг во всей красе, добро пожаловать в новый и прекрасный мир
Ответить | Правка | Наверх | Cообщить модератору
Ответы: #2, #5

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

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

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

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

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

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

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

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

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

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

11. Сообщение от Аноним (11), 30-Янв-26, 13:27   +2 +/
К сожалению, полноценный ревью сгенерированного ллм кода лишает смысла использование ллм в целом из-за приближения затрат на ревью без ллм кода, созданного ллм, к затратам на написание кода и ревью без ллм. Поэтому стоит ожидать, что какое-то количество мертвого кода, добавление которого концептуально свойственно всем 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   –9 +/
Пф, писаки FFmpeg открыли рот чтобы кого-то критиковать?
Чуваки, вы ж не единственные разработчики мультимедийных пакетов.
Или принимайте патчи как удобно настоящим разработчикам, или останетесь вообще без патчей.
Ответить | Правка | Наверх | Cообщить модератору
Ответы: #16, #33, #52

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

32. Сообщение от kravich (ok), 30-Янв-26, 13:53   +1 +/
Ок, возможно.
Но по прежнему сомнительно, что выразительности промптов достаточно для того, чтобы сделать все возможные правки, которые могут потребоваться
В общем, поживем-увидим
Ответить | Правка | Наверх | 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   +4 +/
А что не так?
Если тебе нужно оценить сверху макс скорость которую разовьет белка падая после скольжения по абсолютно ровному столу.....

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

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

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    Скрыто ботом-модератором–2 +/
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #39 Ответы: #73

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

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

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

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

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

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

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

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

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

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

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

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

Шогготов?

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

50. Сообщение от ХрюХрю (?), 30-Янв-26, 16:05   +6 +/
ничего вы не понимаете!
надо const int BOCEMb = 8;
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #4

51. Сообщение от Аноня (?), 30-Янв-26, 16:06   +/
А что в них плохого сейчас ?
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #5 Ответы: #67

52. Сообщение от Аноня (?), 30-Янв-26, 16:07   +1 +/
Настоящие разработчики не пишут код ?
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #13

53. Сообщение от Аноня (?), 30-Янв-26, 16:09   +/
Ну может чел вообще первый коммит в своей жизни делал. Скажите спасибо что он свою биографию туда не добавил )
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #47

54. Сообщение от онанист (?), 30-Янв-26, 16:14   +2 +/
это чтобы в военное время заменить на 14
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #4

55. Сообщение от Аноним (37), 30-Янв-26, 16:17   +/
Не каждый вклад нормально переваривается.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #44

56. Сообщение от Смузихлеб забывший пароль (?), 30-Янв-26, 16:37   +/
xxx :)
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #42

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

58. Сообщение от Аноним (58), 30-Янв-26, 16:41   +/
> если у белки есть шанс

Вот именно. Ты этого никогда не узнаешь, если будешь значения подбивать к своей любимой десятичной системе счисления, а не вычисляя точное значение.

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

59. Сообщение от Аноним (60), 30-Янв-26, 16:43   +/
еще не факт, что это ИИ. Я вполне естественный "интеллект" видел, который воспринял "no magic numbers please" буквально и притащил "рефакторинг" вида const int THIRTY = 30.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #4

60. Сообщение от Аноним (60), 30-Янв-26, 16:45   +/
имена нормальные, а вот 5 уровней вложенности на два экрана - бывает, конечно, что иначе никак, но обычно можно подразбить
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #57

61. Сообщение от laindono (ok), 30-Янв-26, 16:56   +/
Яб добавил, что есть нюансы. Например если лексическое время жизни счётчика большое (длинный цикл), то я могу заменить i на index. Если эти счётчики имеют геометрический смысл, то заменю на x, y, z. И так далее. Но это эвристики, а не правила. Можно применять для себя, но нельзя заставлять других.

Длинные имена в данном контексте могут ухудшать читабельность кода.

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

62. Сообщение от похнапоха (?), 30-Янв-26, 17:11   +/
Просто корпорации этот проект не интересен, поэтому можно делегировать любого котрибьютора. Вот если бы это был не открытый проект мультимедийного проекта, а какой-нибудь коммерческий проект по заказу Министерства обороны - тогда бы не и отношение было бы иным, и работали бы специалисты более высокой квалификации.
Ответить | Правка | Наверх | Cообщить модератору

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

64. Сообщение от fidoman (ok), 30-Янв-26, 17:23   –1 +/
оптимизации плевать на сегменты, да и будет она в нём, только если её в глобальной области определить, а не в функции - тогда она будет в стеке.
А оптимизация в любом случае const использует, хотя в памяти, занятой этой переменной может быть любое значение

#include <stdio.h>
int main()
{
    const int a=5;
    int *b=(int*)&a;
    *b=10;
    const int *c=&a;
    printf("a=%d [%p] b=%d [%p] c=%d [%p]\n", a, &a, *b, b, *c, c);
}

a=5 [0x8207b3d0c] b=10 [0x8207b3d0c] c=10 [0x8207b3d0c]

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

65. Сообщение от НЕТ (?), 30-Янв-26, 17:30   +/
относится подготовке
2 ошибки. Повнимательнее относитесь к подготовке новости.
Ответить | Правка | Наверх | Cообщить модератору

66. Сообщение от fidoman (ok), 30-Янв-26, 17:30   +/
посчитайте сумму углов треугольника, приравняв значение 𝜋 к трём.
чем не задачка на недекартову геометрию.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #18

67. Сообщение от Аноним (67), 30-Янв-26, 17:34   +/
цена, а что хорошего?
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #51

68. Сообщение от premium user (?), 30-Янв-26, 17:36   +1 +/
Щёчики не нужны, часто можно указатели на данные использовать вместо них, это экономит такты ЦП, регистры, и нейминг очевиден. (Pixel *pix = data, *pix_max = data + size; pix < pix_max; pix++) { pix->r = ...; }
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #15

69. Сообщение от pkdr (ok), 30-Янв-26, 17:38   +/
Я как физик по образованию ничего плохого в этом не вижу, да чаще всего в физике нужны и используются 2 значащие цифры числа, поэтому чаще всего используется именно значение g как 9,8. Но в огромном числе реальных физических расчётов вообще достаточно первой значащей цифры (а иногда вообще порядка), в таких случаях как раз g будет равно 10.

Ну и да в большинстве реальных бытовых случаев и для любых оценочных расчётов как раз смело можно считать g равным 10, если тебе не нужна погрешность больше пары-тройки процентов, то 9.8 тебе и не надо.

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

70. Сообщение от Аноним (70), 30-Янв-26, 17:48   +/
Теперь понятно, откуда в C-программах берутся CVE: их корпы туда вот так вставляют!
Ответить | Правка | Наверх | Cообщить модератору

71. Сообщение от Аноним (71), 30-Янв-26, 17:50   +/
Комплектные постпроцессоры CAM-системы на работе написаны на своем языке и выглядят именно так. Реально one = 1, two = 2 и т.д.
Ответить | Правка | Наверх | Cообщить модератору

72. Сообщение от Аноним (14), 30-Янв-26, 17:59   +1 +/
И чего ты просто написал овнокод с магическими переменными. И даже в нём уже невозможно разобраться. Автор сабжевого патча хотя бы в именование переменных умеет четкое. И он явно указывает что ждёт от конста, а не надеется на поведение компилятор в конкретном случае.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #64

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


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

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




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

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