The OpenNET Project / Index page

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



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

Оглавление

Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..., opennews (??), 23-Ноя-18, (0) [смотреть все]

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


38. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +1 +/
Сообщение от M i Memail (?), 23-Ноя-18, 23:46 
Глянул.
Какой-то трэш и каша.
Комментарии и сообщения в перемешку на английском и на русском.
Весь код свален в одну кучу:
https://github.com/hakavlad/nohang/blob/master/nohang
Ответить | Правка | Наверх | Cообщить модератору

39. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +1 +/
Сообщение от M i Memail (?), 23-Ноя-18, 23:57 
И да, ни одного теста не увидел.
Пожалуй, я бы побоялся такое запускать, даже если бы оно мне было нужно.

Но справедливости ради хочу похвалить за старание.
Надеюсь, следующие проекты будут отличаться в лучшую сторону.

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

40. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  –3 +/
Сообщение от Нодежээсовцы и питонисты (?), 24-Ноя-18, 00:02 
Ну если ты не понимаешь что-то это не значит что написано плохо. Писать юникс-демоны - задача сложная. Код сложный потому что тема сложная.

Linux, nohang, X.Org Server... у всех этих наших проектов высок порог вхождения, мы пишем системные вещи.

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

41. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +1 +/
Сообщение от M i Memail (?), 24-Ноя-18, 00:05 
Код там не сложный, но его качество оставляет желать лучшего.
Ответить | Правка | Наверх | Cообщить модератору

57. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  –1 +/
Сообщение от Orduemail (ok), 24-Ноя-18, 05:55 
> Весь код свален в одну кучу:

Там всего 1.5k строк, включая комменты и пустые строки, кои по моему в коде используются излишне часто. Если бы там было 5-10k можно было бы поныть, что слишком много в одном файле, а так...

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

84. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Аноним (84), 24-Ноя-18, 16:25 
Это уже очень много. Есть принцип единой ответственности, который явно нарушается.
Ответить | Правка | Наверх | Cообщить модератору

92. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  –1 +/
Сообщение от Orduemail (ok), 24-Ноя-18, 19:48 
> Это уже очень много. Есть принцип единой ответственности, который явно нарушается.

Знаешь, прежде чем сувать эти принципы куда-нибудь, надо посмотреть насколько они применимы. Я бы рекомендовал тебе открыть учебник снова, и перечитать его _внимательно_, обращая особое внимание на границы применимости принципов и на то, какие проблемы эти принципы призваны решать, и соответственно когда следование этим принципам осмысленно. То есть, я понимаю, что для сдачи экзамена ты заучивал наизусть названия и формулировки принципа. Но в реальной жизни, важнее границы применимости, чем названия.

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

103. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от zoonman (ok), 25-Ноя-18, 08:13 
># found experimentally
>zram_disksize_factor = 0.0042

Магические константы, забытые комменатрии, отсутствие какой-либо структуры. Код требует шлифовки.
Вообще его бы переписать на Go, работал бы быстрее и память меньше кушал.
Считаю текущий вариант прототипом и в продакшене такое бы использовать не стал.
Советую Алексею привести код в порядок, добавить тесты и бенчмарки.

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

111. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Orduemail (ok), 25-Ноя-18, 14:40 
>># found experimentally
>>zram_disksize_factor = 0.0042
> Магические константы, забытые комменатрии, отсутствие какой-либо структуры. Код требует
> шлифовки.

Я к любому коду могу придраться, и сказать ой, фу, тут не то, а здесь не так. К любому. Но я, в отличие от некоторых, не склонен хвастаться этим, я не считаю это сколь-нибудь значимым свойством моей личности.

> Вообще его бы переписать на Go, работал бы быстрее и память меньше
> кушал.

Перепиши, тебе никто не мешает.

> Считаю текущий вариант прототипом и в продакшене такое бы использовать не стал.

Чтобы заиметь такое весомое мнение вовсе не обязательно было читать код, достаточно было заголовка новости, в нём чётко указано, что версия софтины 0.1.

> Советую Алексею привести код в порядок, добавить тесты и бенчмарки.

Надеюсь, что Алексей пошлёт тебя с твоими советами по известному адресу. Я всегда подобных советчиков посылаю, следуя заветам Торвальдса: либо код на бочку, либо засуньте свои советы себе туда, куда солнышко не заглядывает. Слишком уж много советчиков развелось в интернете.

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

113. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Xasd (ok), 25-Ноя-18, 15:00 
> в нём чётко указано, что версия софтины 0.1.

это много или мало?

если там было бы написано что версия софтины 345.0 (а не 0.1) -- это говорило бы о чём-то другом?

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

114. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Orduemail (ok), 25-Ноя-18, 15:10 
>> в нём чётко указано, что версия софтины 0.1.
> это много или мало?
> если там было бы написано что версия софтины 345.0 (а не 0.1)
> -- это говорило бы о чём-то другом?

Да.

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

129. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от zoonman (ok), 25-Ноя-18, 23:12 
Это такой способ сделать тег. Поставил первый попавшийся тег, v0.1 показался нормальным.
Человек знакомый с версионированием сделал бы 0.0.1-alpha.
Т.к. это самая настоящая альфа.
Ответить | Правка | К родителю #113 | Наверх | Cообщить модератору

133. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Orduemail (ok), 26-Ноя-18, 02:59 
> Это такой способ сделать тег. Поставил первый попавшийся тег, v0.1 показался нормальным.
> Человек знакомый с версионированием сделал бы 0.0.1-alpha.
> Т.к. это самая настоящая альфа.

Бла-бла-бла. Ещё один начитавшийся учебников.

Как много тебе известно примеров программ, которые бы стояли в продакшне с версией меньше чем 0.5? Вот совершенно не важно, какой схемы версионирования придерживаются разработчики, лишь бы версия была бы меньше, чем 0.5, и стояла бы в продакшне.

Я не понимаю о чём спор? О том, что номер версии не несёт никакой информации? Если так, то вы ничего не понимаете в информации. Открой Thinking Bayes (в гугле находится легко), и найдите там задачку про номер поезда. Суть её в следующем: мы, проходя мимо ж/д путей, увидели локомотив принадлежащий компании X, на этом локомотиве был порядковый номер, допустим, 37. Вопрос: сколько всего локомотивов у компании X? Если тебе кажется, что самый точный ответ на этот вопрос "не меньше 37", то я повторю свой совет: открой Thinking Bayes и почитай, причём в этом случае лучше сначала и до конца. А прочитав, осознай, что голова человека всегда thinking bayes, просто в большинстве случаев не используя формул, распределений вероятностей и python'а, а на голом автоматизме, нейроны в голове сплетены таким образом. Школьное образование и в первую очередь школьная математика внушают нам что всё это туфта, и логика -- единственный вид мышления, в реальности же логика вообще не мышление, а лишь "unit-testing" для мышления, с ограниченной сферой применимости. В реальности мозги работают не так, как представляли себе древние греки.

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

140. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +1 +/
Сообщение от none_first (ok), 26-Ноя-18, 14:36 
https://greenteapress.com/wp/think-bayes/ если про эту книгу - то название несколько другое ;)
Ответить | Правка | Наверх | Cообщить модератору

148. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от zoonman (ok), 26-Ноя-18, 19:45 
> Бла-бла-бла. Ещё один начитавшийся учебников.

Пора кому-то повзрослеть. Учебники написаны на основе реального опыта других людей.
Они написаны людьми, которые давно прошли стадию "совершенно не важно, какой схемы версионирования придерживаются разработчики, лишь бы версия была бы меньше, чем 0.5, и стояла бы в продакшне", наломали дров и набили шишек, поняли, что они делали не так и постарались закрепить это в форме учебников.
Да, современное поколение любит х**к-х**к и в продакшен и пофиг что там. Большинство из таких не задерживается на работе больше года и прыгает на другую "галеру".
А потом за ними приходят люди, которые делают все по учебнику. Делают долго и занудно, после чего все "просто работает". Обычно после этих "школяров" разного рода OOMKillers не требуются. У них память не течет, поскольку все сделано по учебнику, и системы масштабируются автоматически, поскольку опять же все сделано по учебнику.
Вобщем, я не заинтересован в продолжении этой дискуссии. Вам нужен опыт. Получите его, начнете меня понимать.

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

150. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Orduemail (ok), 26-Ноя-18, 22:55 
>> Бла-бла-бла. Ещё один начитавшийся учебников.
> Пора кому-то повзрослеть.

Да, несомненно.

> Учебники написаны на основе реального опыта других людей.

Но учебник -- это не опыт. Даже если это хороший учебник, то чтение его не заменяет опыта. Учебник, да и вообще всякие образовательные курсы, в том числе и в/о из вуза -- это лишь база, фундамент на котором можно строить свой опыт. Сам по себе фундамент без опыта бесполезен. И таймлайн примерно такой: человек читает учебники в вузе, после этого он приходит работать. Года три он работает следуя принципам из учебников, и после этого он становится хорошим специалистом. А после этого, _некоторые_ могут пойти дальше, поняв что _любой_ абсолютно принцип, и не только в программировании, _любой_ абсолютно теоретический закон имеет границы применимости. Что любая теория имеет границы применимости, и что, по-хорошему, начинать изучать любую теорию следует с изучения границ применимости. И если человек, заявляющий, что он знает какой-то принцип, не может сходу привести примеров ситуации, когда этот принцип неприменим, то этот человек не знает этого принципа.

> Они написаны людьми, которые давно прошли стадию "совершенно не важно, какой схемы
> версионирования придерживаются разработчики, лишь бы версия была бы меньше, чем 0.5,
> и стояла бы в продакшне", наломали дров и набили шишек, поняли,
> что они делали не так и постарались закрепить это в форме
> учебников.

То есть примера ты привести не можешь? Будешь разводить демагогию, вставать в красивую позу "я владею сакральным знанием недоступным посвящённым, поэтому даже обосновывать слов своих не буду, и не заинтересован в продолжении дискуссии", и вообще всячески пытаться свалить из дискуссии с видом победителя? Ну-ну. Продолжай в том же духе.

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

130. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от zoonman (ok), 25-Ноя-18, 23:16 
Торвальдс уже давно не тот: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/lin... ;-)
Ответить | Правка | К родителю #111 | Наверх | Cообщить модератору

134. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Orduemail (ok), 26-Ноя-18, 03:00 
> Торвальдс уже давно не тот: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/lin...
> ;-)

Вот к чему ты это сказал? Во-первых, не то чтоб давно, во-вторых, мне-то что? Если Торвальдс принимает Code of Conduct, меня это разве к чему-то обязывает? Или заветы Торвальдса перестали быть заветами Торвальдса?

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

137. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от нах (?), 26-Ноя-18, 12:31 
> Если Торвальдс принимает Code of Conduct, меня это разве к чему-то обязывает?

нет, если ты не собираешься присылать ему патчи. Если патч, исправляющий критическую ошибку в ядре, будет содержать коммент вида "афтар - макака, убейте его кто-нибудь", его могут выкинуть и забить на исправления.

Но потреблятелей эта проблема, действительно, не должна беспокоить, вы ведь никаких патчей никому не присылаете?

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

138. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Orduemail (ok), 26-Ноя-18, 13:37 
Ты о чём вообще, родный? Ты тред читал? Я сказал о том, что я следую завету Торвальдса: либо код на бочку, либо до свидания. При чём здесь потребляти, отправление патчей _торвальдсу_, маты в комментах и CoC? Твоя больная тема? Увидев её упоминание никак не можешь сдержаться и не высказаться? Против этого я могу предложить копинг стратегию: купи подушку, и каждый раз столкнувшись с упоминанием этой темы, кричи в подушку, пока не отпустит. Подушка заглушит звуки, и соседи тебя не убьют тогда за нарушение тишины и спокойствия.
Ответить | Правка | Наверх | Cообщить модератору

142. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Michael Shigorinemail (ok), 26-Ноя-18, 17:30 
Коллеги, прям озадачили.
Ответить | Правка | Наверх | Cообщить модератору

143. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от нах (?), 26-Ноя-18, 17:59 
> Ты о чём вообще, родный? Ты тред читал? Я сказал о том,
> что я следую завету Торвальдса: либо код на бочку,

ты топ-топ менеджер линукса или сравнимого по размеру проекта? (Санитары! Этого в палату к Наполеону и Цезарю) Нет? Тогда какой тебе код, кто его тебе понесет?

А если ты свой код на _чужую_ бочку - то вот теперь привыкай правильно именовать в нем переменные, без расовой дискриминации, и правильно комментировать, не оскорбляя альтернативно-одаренных.

Иначе его просто выбросят в помойку не читая, а тебя зобанят.

Если код из одной строчки и занял пол-часа на ее поиск - то, конечно, вполне можно соблюсти. А потом удивляемся, чего это независимых коммитеров даже в ядро, где полно совсем простых и тривиальных мест, полтора процента, остальные - корпорации.

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

149. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Orduemail (ok), 26-Ноя-18, 21:33 
>> Ты о чём вообще, родный? Ты тред читал? Я сказал о том,
>> что я следую завету Торвальдса: либо код на бочку,
> ты топ-топ менеджер линукса или сравнимого по размеру проекта?

Нет, но и тем не менее я советую даже разработчикам небольших и даже прям скажем маленьких проектов типа NoHang поступать также. Советчиков посылать далеко и надолго.

> Тогда какой тебе код, кто его тебе понесет?

Не знаю кто, но советы точно не нужны.

> А если ты свой код на _чужую_ бочку - то вот теперь
> привыкай правильно именовать в нем переменные, без расовой дискриминации, и правильно
> комментировать, не оскорбляя альтернативно-одаренных.

Ты тупой? Простите за мой французский, вырвалось. Речь идёт о советчиках и коде. Советчиков надо посылать, с теми же кто принёс код можно работать. При чём здесь "правильно именовать переменные", "расовая дискриминация", "правильные комментарии", "оскорбление альтернативно-одарённых"? Я ещё раз повторю совет, чтобы справится с эмоциями при столкновении с болезненной для тебя темы, возьми подушку и прокричись в неё. И да, другой совет тоже повторю: перечитай тред.

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

145. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Аноним (145), 26-Ноя-18, 19:09 
>zram_disksize_factor = 0.0042

Описание есть в начале файла.

>забытые комменатрии

Где?

>отсутствие какой-либо структуры

Структура есть, код разделен на разделы, разделенные строками ############################, и разделы прокомментированы.

>Вообще его бы переписать на Go, работал бы быстрее и память меньше кушал.

Уже. Старт дан. https://github.com/hakavlad/nofreeze

>Считаю текущий вариант прототипом

как вам угодно. Тем не менее, прототип прекрасно работает.

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

146. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от zoonman (ok), 26-Ноя-18, 19:24 
Вот целый кусок забытых комментариев
https://github.com/hakavlad/nohang/blob/master/nohang#L934

Я не хочу придираться к деталям. Просто бросилось в глаза.

У вас хорошая идея, продолжайте ее развивать.

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

147. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Аноним (145), 26-Ноя-18, 19:29 
Это прям свежий коммит, не от автора. Будет почищено вскоре. Ничего не забыто.
Ответить | Правка | Наверх | Cообщить модератору

152. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Аноним (152), 28-Ноя-18, 17:56 
Говнокод ещё тот

# НА МЕСТЕ!!!
if 'swap_min_warnings' in config_dict:
    swap_min_warnings = config_dict['swap_min_warnings']

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

153. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Аноним (145), 29-Ноя-18, 05:14 
Что Вас смутило в этом участке кода?
Ответить | Правка | Наверх | Cообщить модератору

163. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Аноним (-), 30-Ноя-18, 08:40 
> Что Вас смутило в этом участке кода?

Ну вот меня например такой комент в коде смущает. Видя такое я как-то сразу прикидываю что и остальное каКчество кода оправдает самые смелые ожидания.

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

162. "Выпуск Nohang 0.1, предотвращающего OOM в пространстве польз..."  +/
Сообщение от Аноним (-), 30-Ноя-18, 08:38 
> Комментарии и сообщения в перемешку на английском и на русском.

Это ты еще код bitmessage читать не пробовал. Сразу видно - крутой про огреб крутую идею и подумал как же ее реализовать, если прогать не умеешь? Ну увидел бэйсик или что там вместо него теперь и накодил как умел. Заодно и програмить поучился как раз.

О том что проект надо майнтайнить, в входных данных может быть левак, а огроменный рантайм писаный как курица лапой неважно годится для системного програмизма - такому гению от софтостроя никто никогда не рассказывал конечно. Поэтому он презрительно смотрит на остальных - весь мир у его ног, падите ниц, жалкие червяки! Ведь этот голимый погонщик слонов наконец чего-то смог. Правда, как слон устроен он понятия не имеет, да и повадки он знает лишь целых 5 минут. Так что если слон вдруг взбрыкнет, плюс-минус пара раздавленых зрителей сущая фигня.

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

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

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




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

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