хорошо, через часа 2 сделаю чтобы удобно было делать кодеревью

как обещал, правда чуть позже, рефакторинг не делал https://github.com/armicron/OctoPrint-PrintHistory/commit/02d485187ce12713fa634465c6eeecd10bff7d70

уже вижу что метод parse везде одинаковый, надо выносить в родительский класс

как обещал, правда чуть позже, рефакторинг не делал https://github.com/armicron/OctoPrint-PrintHistory/commit/02d485187ce12713fa634465c6eeecd10bff7d70

так это опять коммит, а не мёрдж-реквест

сейчас исправлю

почитай github flow, кстати. если ещё не

https://github.com/armicron/OctoPrint-PrintHistory/pull/1

почитай github flow, кстати. если ещё не

я открыл вкладку, а почитать пока нет времени

ребзя, привет всем

добрейшего вечерочка

хотел вот узнать, у меня почему то tornado не выполняет set_cookie()

вот я не понимаю

это супер тривиальный вариант, но большего не надо, поэтому, прошу подсказать именно почему set_cookie не работает

причем, даже интереснее

для первой строчке в файле - все работает

для второй - уже нет

в чем беда - не понимаю

В иф то заходит?

а чего гадать. ipdb/wdb-шнись туда и посмотри, @rmnff

В иф то заходит?

да, все условия выполняются, а вот сам self.set_cookie() работает только для первого пользователя в файле

тот кто на первой строке

менял местами с другими пользователями

работает всегда для того, кто первый в строке

а находит пользователей на любой строке

ты понимаешь как куки работают?

ты понимаешь как куки работают?

нет..

насколько я знаю, сохраняют в браузере ключ:значение

так?

вот собственно сам файлик с данными

если местами строчки поменять, то будет работать для rmnff

не меняешь, работает для pidor

нет..

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

и не именуй так ничего, ever. Я про обсценную/не значащую лексику naming matters

@armicron так, теперь с тобой давай разберёмся. Ты читал https://github.com/thoughtbot/guides/tree/master/code-review?

нет, не читал

Be grateful for the reviewer's suggestions. ("Good call. I'll make that change.")

@armicron поревьюил бегло по стилю

по логике/перформансу лень :) мб кто другой набежит ещё

спасибо, там и так 30 комментариев

спасибо, там и так 30 комментариев

Можешь autopep8 натравить) или flake8 ;)

pep8 ничего не покажет, потому что я им проверял

Хотя пичарм и так все говорит...

Нормас ..

Можешь autopep8 натравить) или flake8 ;)

про flake8 один из каментов =)

дрочба на пеп8

что может быть лучше

только кокаин с соска мулатки поутру, конечно же

ну и троллить лига за эклипс разве что

Хотя пичарм и так все говорит...

увы. Помню парень перешёл как-то по моему настоянию с атома на пайчарм и страдал, что пайчарм: а) линтит как-то по-своему, не по pep8 б) линтит и форматирует различно — это вообще мозг ломает

увы. Помню парень перешёл как-то по моему настоянию с атома на пайчарм и страдал, что пайчарм: а) линтит как-то по-своему, не по pep8 б) линтит и форматирует различно — это вообще мозг ломает

это изменяемое поведение, в настройках можно переключится на пеп8

увы. Помню парень перешёл как-то по моему настоянию с атома на пайчарм и страдал, что пайчарм: а) линтит как-то по-своему, не по pep8 б) линтит и форматирует различно — это вообще мозг ломает

К сожалению, там в редакторе баги до сих пор появляются... Недавно эпичный баг там словил. Автоформат для регулярки применил, строка раскопировалась четыре раза. До этого встречались проблемы с переносом комментариев, с отступами. Еще эпичная бага была с дебаг точкой на второй перенесенной строке. Такая точка просто не срабатывала. А так, привык уже просто...

при всей моей пылкой юношенской любви к джейтбрейнз, там дофига проблем

это изменяемое поведение, в настройках можно переключится на пеп8

Который оно тоже реализует по своему. Это не pep8.py и не autopep8 и их не подключить туда.

IDE самоудовлетворяющееся

при всей моей пылкой юношенской любви к джейтбрейнз, там дофига проблем

да где их нет... Любой умный инструмент всегда продуцирует проблемы. Чем умнее - тем больше проблем

пишу в vim, не знаю проблем

потому что выйти не можешь? :)

(да)

??

ходят слухи, что существует возможность загрузить емакс напрямую из едра. И получить ОС emacs. Вот это я понимаю - хардкор

ERROR: S client not available

поделись ".vimrc" плиз.

ходят слухи, что существует возможность загрузить емакс напрямую из едра. И получить ОС emacs. Вот это я понимаю - хардкор

И питон можно. Только незачем.

ходят слухи, что существует возможность загрузить емакс напрямую из едра. И получить ОС emacs. Вот это я понимаю - хардкор

Emacs таки лучше с X. Писать код в терминале не слишком удобно.

Зависит от шрифта в терминале

Как раз про шрифты: "It's fulltime job to get font rendering in X that doesn't make you want to actively poke your eyes out"

Как раз про шрифты: "It's fulltime job to get font rendering in X that doesn't make you want to actively poke your eyes out"

Это откуда-то из прошлого заявление. И то это было раз и навсегда. А теперь еще и сразу из коробки. Кстати, в Linux сейчас из коробки рендеринг шрифтов получше маковского будет. По крайней мере, в Gnome на Fedora. Не знаю как там у космонавта.

Это откуда-то из прошлого заявление. И то это было раз и навсегда. А теперь еще и сразу из коробки. Кстати, в Linux сейчас из коробки рендеринг шрифтов получше маковского будет. По крайней мере, в Gnome на Fedora. Не знаю как там у космонавта.

После freetype 2.7 лучше стало, но всё равно это слишком много боли под каждый монитор настраивать шрифты

По поводу макоси весьма спорно, но вот винда точно из коробки по рендерингу шрифтов проигрывает

import ConfigParser import io import os ... сделал imports в алфавитном порядке, что-то не очень хорошо выглядит

сначало системные потом внутренние

pycharm не различает

хотя тут все системные

import ConfigParser import io import os ... сделал imports в алфавитном порядке, что-то не очень хорошо выглядит

Это ты еще 80 строк импортов по четырем секциям разбитых не видел. Глаз пока несущественные вещи не научился пропускать...

различает

ну значит это нормальная секция import? мне просто не нравится что длинная строка на 1-м месте

А мне не нравится сортировка импортов по алфавиту. Это разве из pep8? Всегда думал, что это из google code style guide

да, pep8 только рекомендует группировать * standard library imports * related third party imports * local application/library specific imports про сортировку ничего нет

https://medium.com/@squeaky_pl/million-requests-per-second-with-python-95c137af319#.gtthxivwd

какой-то левый бенч

я статью если честно не читал еще, пролистал. но вот тут [1] чувак тоже сомневается, но цифры там всё равно достойные. [1] https://github.com/squeaky-pl/japronto/issues/3

https://medium.com/@squeaky_pl/million-requests-per-second-with-python-95c137af319#.gtthxivwd

Japronto is written almost entirely in C. The parser, protocol, connection reaper, router, request and response objects are written as C extensions.

в response всё равно как-то впинхнуть что-то. :) не из C же он туда пихает

OS was reporting Xeon® CPU E5–2666 v3 @ 2.90GHz

10 ядер, с учётом ht - 20 vcpu. оно двухсокетное, так что в машине скорее всего была их пара.

так что я бы поделил то число(1.2млн) хотя бы на число физических ядер.

а осталльные участники бенчмарка бежали на одном? ))

с hello world у меня pypy показывал 160-170 krps на одном ядре.

ну про HT это только написано 20. по факту там будет 10. :)

These results were obtained on AWS c4.2xlarge instance that has 8 VCPUs