These are chat archives for synrc/n2o

13th
Oct 2015
Namdak Tonpa
@5HT
Oct 13 2015 04:28
а примеры у тебя работают после твоего фикса? (боюсь мерж нажимать)
Namdak Tonpa
@5HT
Oct 13 2015 04:49
а зачем вызывать pid и игнорировать его возврат и отдавать ок ?
n2o_async:pid(Class,Name,undefined), ok.
сомнительный пул реквестик :-)
Andy
@m-2k
Oct 13 2015 04:51
Где ты ок увидел
Namdak Tonpa
@5HT
Oct 13 2015 04:52
m-2k/n2o@44a5fa9
Andy
@m-2k
Oct 13 2015 04:54
Ок не я писал
Namdak Tonpa
@5HT
Oct 13 2015 04:54
:-)
так шо пример работает?
waterline пишется?
чат работает?
плюс ко всему ты арность у pid поменял
не не буду принимать такой пул реквест
зачем ты изменил арность?
зачем ты {get} на 'HNDLR' поменял ?
Namdak Tonpa
@5HT
Oct 13 2015 05:00
ты хочешь чтобы HNDLR оставил свой след в N2O протоколе ? :-)
Andy
@m-2k
Oct 13 2015 05:01
Это ж эрланг вэй
Арность поменял так как много скобок было )
В более отп стиле стало
Namdak Tonpa
@5HT
Oct 13 2015 05:06
ты я вижу протокол n2o вообще не понимаешь раз у тебя возникла мысль атом назвать 'HNDLR'
Andy
@m-2k
Oct 13 2015 05:07
Да эт к протоколу мало относится, больше к костылю
Ну ты сформулируй чо да как, я пофиксаю
Namdak Tonpa
@5HT
Oct 13 2015 05:08
ну можно было например не выйобываться и ничего у меня не переименовывать
а просто пофиксать финализацию на стопах
то про что шла речь
нахуя переписывать было все
ничего не трогай поменять 1 строчку только и сделай PR
{get} это универсальный протокол получения стейта процесса
для BPE это #process
для async это #handler
это не костыль :-)
арность пида тоже не надо менять
просто пофиксай одну проблему
и опиши ее
одна проблема — один PR — одна строчка
Andy
@m-2k
Oct 13 2015 05:41
Ггг ок
Namdak Tonpa
@5HT
Oct 13 2015 05:42
а то мой мозг с утра не готов читать 'HNDLR'
это откуда вообще ты что под Win32 писал?
Andy
@m-2k
Oct 13 2015 05:49
MFC :smile:
Namdak Tonpa
@5HT
Oct 13 2015 05:51
реально или шутишь?
Andy
@m-2k
Oct 13 2015 05:51
реально конечно же
Namdak Tonpa
@5HT
Oct 13 2015 05:51
Win32 же лучше
:-)
WTL
ATL
у меня такое ощущение что ты не понимаешь что ты делаешь
init(Pid,Class,Name) when is_pid(Pid) -> n2o_async:pid(Class,Name,Pid), send(Pid,{parent,self()}).
в этой функции опять pid вызывается но нахуя не понятно :-)
Andy
@m-2k
Oct 13 2015 05:53
get/set
get/2 set/3
Namdak Tonpa
@5HT
Oct 13 2015 05:53
я попросил тебя одну строчку поменять
а не семантику и арность вызовов :-)
Andy
@m-2k
Oct 13 2015 05:53
5 сек
Namdak Tonpa
@5HT
Oct 13 2015 05:54
а это тоже самое
ты просто закрыл
блядь
а я думал это шото новое
:-)
Andy
@m-2k
Oct 13 2015 05:54
лалка
Namdak Tonpa
@5HT
Oct 13 2015 05:59
зачем ты создал эту функцию
pid({Class,Name},Pid) -> wf:cache({Class,Name},Pid). ?
почему просто не написать wf:cache ?
ты можешь просто сделать комит на 1 строчку или это для этой задачи невозможно ? :-)
нахуя ты функции придумываешь ? :-)
несуществующие
Andy
@m-2k
Oct 13 2015 06:01
а какая логика в том что pid/1 это getter, а устанавливаешь значения через setter wf:cache/2?
пиздец же
половинная абстракция
Namdak Tonpa
@5HT
Oct 13 2015 06:02
где ты учился
шо за понятия половинная абстракция :-)
ты хотел сказачть что скучаешь по симметричным функциям?
так это не они :-)
я не приму это
очень плохо что ты меня не слышишь
Andy
@m-2k
Oct 13 2015 06:03
ты wf:cache/1 переопределил как pid/1
Namdak Tonpa
@5HT
Oct 13 2015 06:03
а не в том что ты что-то там делаешь
Andy
@m-2k
Oct 13 2015 06:03
нахуя?
Namdak Tonpa
@5HT
Oct 13 2015 06:03
я тебя попросил сделать точечный комит
а ты опять функции выдумываешь и код переписываешь
вот тоже надо убрать
но это ведь не касается этой задачи
задача то в чем?
пофиксать стоп
а не бютифаить код
Andy
@m-2k
Oct 13 2015 06:04
why not
Namdak Tonpa
@5HT
Oct 13 2015 06:05
потому что я тебя попросил с заворотом комита
а ты просто проигнорировал мою просьбу
и дальше комитишь шо хочешь
Andy
@m-2k
Oct 13 2015 06:05
так ты только материшься и засираешь все вокруг
без объяснений
как тут можно понять что ты хочешь
Namdak Tonpa
@5HT
Oct 13 2015 06:06
ну вот ты наверно хочешь специально чтобы я нервничал
я же попросил сделать точечный комит по задаче
потом это все читать невозможно
приучайся к культуре немножко
хватит гитхаб анархии
насколько я понял из комита
"{Class," если вот это добавить в код
то все станер работать
т.е. это коммит 7 байт
Andy
@m-2k
Oct 13 2015 06:09
ты не правильно понял
Namdak Tonpa
@5HT
Oct 13 2015 06:09
я хочу видеть точечные комиты
если ты хочешь переписать ты можешь переписать но другим комитом
пока я хочу понять какой минификс фиксает проблему
Andy
@m-2k
Oct 13 2015 06:10
ДА ПОНЯЛ Я
Namdak Tonpa
@5HT
Oct 13 2015 06:11
код я сам смогу наилучшим образом отрефакторить
за это ты не переживай
ненада константы менять {get} хаком называть и код реорганайзить
:-)
Namdak Tonpa
@5HT
Oct 13 2015 06:16
я все равно сомневаюсь что это минимальный фикс но смержил :-)
Andy
@m-2k
Oct 13 2015 06:16
минимальный
Alex Popov
@aseidlitz
Oct 13 2015 09:05
I am trying to follow example from roster's readme and getting the following error:
6> {ok,U} = roster:create_user("user1","Test","User"). 
** exception error: bad argument
     in function  apply/3
        called as apply([],next_id,[])
     in call from kvs:next_id/3 (/Users/alexp/Proj/Erlang/EasySim/poc/deps/kvs/src/kvs.erl, line 227)
     in call from roster:create_user/3 (/Users/alexp/Proj/Erlang/EasySim/poc/deps/roster/src/roster.erl, line 14)
Any tips on what's wrong?
Namdak Tonpa
@5HT
Oct 13 2015 09:12
please ensure you have set application:get_env(kvs,dba)
Alex Popov
@aseidlitz
Oct 13 2015 09:13
I see... it returns undefined.
Namdak Tonpa
@5HT
Oct 13 2015 09:14
it should be store_mnesia
I should patch KVS for this
Alex Popov
@aseidlitz
Oct 13 2015 09:16
Strange, though. My sys.config contains {kvs,[{dba, store_mnesia}, {log_modules,roster}, {schema, [kvs_user,kvs_acl,kvs_feed,kvs_subscription,roster]}]}].
Namdak Tonpa
@5HT
Oct 13 2015 09:16
odd parens
Alex Popov
@aseidlitz
Oct 13 2015 09:18
this is copy/paste issue... I left out n2o part of config. The whole conf is
[{n2o,[{log_modules,roster},{port,8000},{route,roster_routes}]},
 {kvs,[{dba, store_mnesia},
       {log_modules,roster},
       {schema, [kvs_user,kvs_acl,kvs_feed,kvs_subscription,roster]}]}].
Namdak Tonpa
@5HT
Oct 13 2015 09:19
then ensure your sys.config is readed and kvs is started
Alex Popov
@aseidlitz
Oct 13 2015 09:20
that's it. I don't have kvs running...
Namdak Tonpa
@5HT
Oct 13 2015 09:21
do you have .applist ?
Alex Popov
@aseidlitz
Oct 13 2015 09:22
yes.
$ cat .applist [kernel,stdlib,ranch,crypto,compiler,syntax_tools,gproc,cowlib,cowboy,n2o,esim,erlydtl,jsone,nitro]
Namdak Tonpa
@5HT
Oct 13 2015 09:22
but there is no kvs
Alex Popov
@aseidlitz
Oct 13 2015 09:22
I didn't run mad plan after adding roster
Namdak Tonpa
@5HT
Oct 13 2015 09:22
now is ok ?
Alex Popov
@aseidlitz
Oct 13 2015 09:23
let me try...
application:get_env(kvs,dba). is still undefined, but kvs is running this time. How do I make sure sys.config is read?
Namdak Tonpa
@5HT
Oct 13 2015 09:24
file:consult
but I see it's ok
application:get_env(kvs,log_modules) ?
application:get_env(n2o,port) ?
Alex Popov
@aseidlitz
Oct 13 2015 09:27
5> file:consult('sys.config'). {ok,[[{n2o,[{port,8001}, {route,routes}, {log_modules,esim}]}]]}
Namdak Tonpa
@5HT
Oct 13 2015 09:28
о
так тут нифига ж нет :-)
Alex Popov
@aseidlitz
Oct 13 2015 09:28
only n2o part of the file is being read. Here's the file iteself:
 {kvs,[{dba, store_mnesia},
       {log_modules,roster},
       {schema, [kvs_user,kvs_acl,kvs_feed,kvs_subscription,roster]}]}].
nda. nifiga...
Namdak Tonpa
@5HT
Oct 13 2015 09:29
так может сис конфиг не там лежит или что
Alex Popov
@aseidlitz
Oct 13 2015 09:30
Да вроде там...
$ ls -aF ./ .DS_Store apps/ erlang.log.1 roster_js.js sys.config ../ .applist deps/ rebar.config run_erl.log vm.args
Ладно, утром посмотрю, мне через 4 часа на работу :-(
Namdak Tonpa
@5HT
Oct 13 2015 09:31
ну полюбому mad читает через file:consult
Alex Popov
@aseidlitz
Oct 13 2015 09:32
Спасибо за помощь!
Namdak Tonpa
@5HT
Oct 13 2015 09:32
поэтому то что он отдает это и берется
это же эрланг OTP :-)
интересно почему не работает видно что-то с путями
где то что-то переопределено
Alex Popov
@aseidlitz
Oct 13 2015 09:40
С фаилом разобрался... file:consult его правильно видит, но application:get_env(kvs,dba) -> undefined.
Namdak Tonpa
@5HT
Oct 13 2015 09:41
так там же была вся секция не видна
Alex Popov
@aseidlitz
Oct 13 2015 09:41
Но я теперь знаю в чем дело, завтра разбирусь... Еще раз спасибо что навел на след.
Теперь видна... Я в едиторе открыл sys.config из deps... Теперь он читается правильно:
8> file:consult('sys.config'). {ok,[[{n2o,[{log_modules,roster}, {port,8000}, {route,roster_routes}]}, {kvs,[{dba,store_mnesia}, {log_modules,roster}, {schema,[kvs_user,kvs_acl,kvs_feed,kvs_subscription, roster]}]}]]} 9> application:get_env(kvs,dba). undefined
Namdak Tonpa
@5HT
Oct 13 2015 09:43
но тут же {dba,store_mnesia}
Alex Popov
@aseidlitz
Oct 13 2015 09:44
Вот я тоже этого не понимаю...
Over and out.