09:09 <foxmask> bonjello
09:20 <jpcw> matin café
09:21 <foxmask> jpcw: l'ami ricoré :)
09:22 <jpcw> haha
09:22 <foxmask> chi viendu avce des pépito à la place des croissasnts :D
16:30 <GMLudo> gawel: suite au bug d'hier, j'ai un bout de code qui ne fait plus faire pipi partout à Panoramisk, par contre, ça fait pêter deux tests. Je n'arrive pas à les réparer. Je peux pusher le commit dans le repo principal, ou je pushe ça dans celui d'Eyepea ?
16:31 <GMLudo> gawel: (histoire d'éviter que travis râle au moment du test)
16:34 <gawel> push chez toi
16:34 <gawel> j'avais une idée, moi aussi
16:34 <gawel> suffit d'avoir des clé dans les deux dict
16:38 <GMLudo> gawel: oui, mais après, y'a le pb du clean-up
16:38 <gawel> bah non, tu pop les deux
16:41 <GMLudo> gawel: le pb, c'est que l'event a soit une référence, soit l'autre, mais pas les deux en même temps. ou alors, il faut utiliser les références dans l'action
16:43 <gawel> la https://github.com/gawel/panoramisk/blob/master/panoramisk/connection.py#L30
16:43 <gawel> tu ajoute self.responses[data.action_id] = data (qui sera equal à data.id dans le cas d'une action)
16:44 <gawel> et t'a résolu le problème
16:44 <gawel> quand tu pop, tu pop les deux pareil
16:47 <gawel> faut rajouter un action_id à Message aussi
16:47 <gawel> mais ça me semble le plus clean
16:51 <GMLudo> gawel: voici le commit: https://github.com/Eyepea/panoramisk/commit/c712edb2379038380a09121596cece5a5fbb7543
17:00 <gawel> j'aimerai autant ma proposition, si ça marche. ça rajouterai moins de code
17:02 <gawel> je check, vite fait
17:10 <gawel> voila, ça marche
17:12 <gawel> GMLudo: https://github.com/gawel/panoramisk/commit/703e92f63ed86b76d22ab284f45c9d8bbb665603
17:12 <gawel> j'ai repris ton test. et ça casse rien :)
22:27 <alain_afpy> ERROR membres(2) HTTPConnectionPool(host='www.afpy.org', port=80): Request timed out. (timeout=10)
22:30 <alain_afpy> INFO membres fixed
22:36 <GMLudo> gawel: c'est cool que les tests passent :-) Par contre, j'aurais un yocto détail à remonter: J'ai switché vers ton code mon daemon en prod, et maintenant ça stackstrace à tous les coups, même dans le cas général où la commande AGI est lancé sur un channel vivant ;-)
22:41 <GMLudo> gawel: https://gist.github.com/GMLudo/30d145e84f7fab38a967