IRC logs of #tryton for Monday, 2010-01-11

chat.freenode.net #tryton log beginning Mon Jan 11 00:00:02 CET 2010
CIA-5vengfulsquirrel roundup * #1372/Performance problem with sorting one2many.:00:18
CIA-5[new] This change reduced a 10 minute delay into a 15 second delay.00:18
CIA-5In my case the ids2 list had a length of around 32,000.00:18
CIA-5http://coderevie ...00:18
CIA-5http://bugs.tryton.org/roundup/issue137200:18
cedkvengfulsquirrel: good00:24
cedkvengfulsquirrel: we must use a kind of cache to search index00:26
cedkvengfulsquirrel: oops, I did not see that you wrote a patch00:28
vengfulsquirrelYeah I'm not sure if there is a faster way to make a list into a dict but that worked for me.00:29
cedkvengfulsquirrel: I made a comment00:32
cedkvengfulsquirrel: but it is false :-)00:33
cedkvengfulsquirrel: I'm still looking00:34
cedkvengfulsquirrel: it is really painful that enumerate return value in this order00:38
vengfulsquirreldict(zip(ids2, range(len(ids2))))  ?00:42
cedkvengfulsquirrel: not sure it is faster00:44
vengfulsquirrelUsing my state of the art datetime.now() performance testing they seem to be the same.  I think actually making it doesn't take that long so maybe the zip would just be prettier?00:45
cedkvengfulsquirrel: zip scale less than: dict((y, x) for x, y in enumerate(a))00:46
cedkvengfulsquirrel: from my test00:47
cedkvengfulsquirrel: with a list of 1000000:47
vengfulsquirrelOh right you mean use a generator list comprehension instead of list comprehension.00:49
cedkvengfulsquirrel: you can propose a patch with my last comment00:49
cedkvengfulsquirrel: yes with list you waste memory00:49
cedkvengfulsquirrel: test script: http://dpaste.com/143420/00:49
cedkvengfulsquirrel: with a list of 1000000 the difference is really big00:51
vengfulsquirrelYeah seems like the generator is the best.00:51
cedkvengfulsquirrel: do you know how to submit patch?00:55
vengfulsquirrelhttp://codereview.appspot.com/186061/patch/1004/500:56
vengfulsquirrelWhat do you mean ?00:56
vengfulsquirrelI updated it00:56
vengfulsquirrelI took out the extra line too, I can add it back though if you wanted it there.00:57
vengfulsquirrelOr I guess I stopped adding the new line.00:57
vengfulsquirrel*blank00:58
cedkvengfulsquirrel: no porblem for the line00:58
cedkvengfulsquirrel: new comment00:59
cedkvengfulsquirrel: by submit patch, I mean in roundup when it will be finished00:59
cedkvengfulsquirrel: to be integrated in hg with your name as author00:59
vengfulsquirrelhttp://code.google.com/p/tryton/wiki/HowtoContribute01:02
vengfulsquirrelI just follow that rihgt ?01:02
vengfulsquirrelI usually mess it up the first 4 times though ha.01:02
cedkvengfulsquirrel: yes01:03
cedkACTION going to sleep01:11
CIA-5vengfulsquirrel roundup * #1372/Performance problem with sorting one2many.: [chatting] Attached a patch01:14
CIA-5http://bugs.tryton.org/roundup/issue137201:14
vengfulsquirrelcedk: Thanks, talk to you tomorrow.01:15
-!- panthera(n=daniel@static.88-198-196-34.clients.your-server.de) has joined #tryton04:07
-!- ChanServ(ChanServ@services.) has joined #tryton04:08
-!- vengfulsquirrel(n=ian@c-69-181-194-95.hsd1.ca.comcast.net) has joined #tryton04:08
-!- udono(n=udono@dynamic-unidsl-85-197-25-39.westend.de) has joined #tryton04:08
-!- yangoon(n=mathiasb@p549F6A91.dip.t-dialin.net) has joined #tryton04:08
-!- juanfer(n=juanfer@190.144.69.234) has joined #tryton04:08
-!- rednul(n=rednul@host-98-127-54-98.bln-mt.client.bresnan.net) has joined #tryton04:08
-!- CIA-5(n=CIA@208.69.182.149) has joined #tryton04:08
-!- fil(n=phil@blue.hands.com) has joined #tryton04:08
-!- sharkcz(n=dan@plz1-v-4-17.static.adsl.vol.cz) has joined #tryton04:08
-!- preC(n=preCTWO@orkan.Informatik.Uni-Oldenburg.DE) has joined #tryton04:08
-!- yangoon(n=mathiasb@p549F7436.dip.t-dialin.net) has joined #tryton05:18
-!- sharoon(n=sharoont@89.243.21.157) has joined #tryton06:41
-!- Timitos(n=timitos@88.217.184.172) has joined #tryton07:33
-!- bechamel(n=user@host-85-201-159-186.brutele.be) has joined #tryton07:39
-!- paepke(n=paepke@p4FEB2AF0.dip0.t-ipconnect.de) has joined #tryton08:16
-!- cedk(n=ced@gentoo/developer/cedk) has joined #tryton08:21
-!- enlightx(n=enlightx@static-217-133-61-144.clienti.tiscali.it) has joined #tryton08:38
-!- mfladischer(n=fladisch@sh018.meduni-graz.at) has joined #tryton09:18
CIA-5paepke roundup * #1350/Error msg on start without python or oo.org:09:56
CIA-5freshly installed win-xp (german)09:56
CIA-5only installed vmware-tools and cause of that windows-installer 3.1.09:56
CIA-5nothing else.09:56
CIA-5maybe there are depend ...09:56
CIA-5http://bugs.tryton.org/roundup/issue135009:56
-!- cedric_b(n=cedric@ANantes-158-1-159-244.w90-59.abo.wanadoo.fr) has joined #tryton11:25
-!- udono(n=udono@dynamic-unidsl-85-197-17-24.westend.de) has joined #tryton12:19
CIA-5Ian Wilson <ian@laspilitas.com> default * 2129:1f45f1707f9e trytond/trytond/model/fields/one2many.py: Add index to improve performance of sorting subsequences for issue137212:21
CIA-5http://hg.tryton.org/trytond/rev/1f45f1707f9e12:21
CIA-5ced roundup * #1372/Performance problem with sorting one2many.: [resolved] Applied12:21
CIA-5http://bugs.tryton.org/roundup/issue137212:21
CIA-5Mathias Behrle <mathiasb@mbsolutions.selfip.biz> default * 540:75d5dca1ac55 account/journal.py:12:25
CIA-5Remove surplus argument for issue 1363.12:25
CIA-5(transplanted from 78254a0c1336465b459882fa1c412c3dabde7254)12:25
CIA-5http://hg.tryton.org/1.4/modules/account/rev/75d5dca1ac5512:25
CIA-5C?dric Krier <ced@b2ck.com> default * 1456:76de1b10567b tryton/tryton/gui/window/view_form/view/list_gtk/parser.py:12:25
CIA-5Add missing import of _LIMIT12:25
CIA-5Add missing window in WinSearch12:25
CIA-5Fix rename go function of WinSearch into run12:25
CIA-5(transplanted from 1ce6ee084c9678de9083be09576c923bcb9b06c8)12:25
CIA-5http://hg.tryton.org/1.4/tryton/rev/76de1b10567b12:25
CIA-5Mathias Behrle <mathiasb@mbsolutions.selfip.biz> default * 492:c30b32b38c6a account/journal.py:12:26
CIA-5Remove surplus argument for issue 1363.12:26
CIA-5(transplanted from 78254a0c1336465b459882fa1c412c3dabde7254)12:26
CIA-5http://hg.tryton.org/1.2/modules/account/rev/c30b32b38c6a12:26
CIA-5C?dric Krier <ced@b2ck.com> default * 1337:ff7421d8d8af tryton/tryton/gui/window/view_form/view/list_gtk/parser.py:12:26
CIA-5Add missing import of _LIMIT12:26
CIA-5Add missing window in WinSearch12:26
CIA-5Fix rename go function of WinSearch into run12:26
CIA-5(transplanted from 1ce6ee084c9678de9083be09576c923bcb9b06c8)12:26
CIA-5http://hg.tryton.org/1.2/tryton/rev/ff7421d8d8af12:26
CIA-5C?dric Krier <ced@b2ck.com> default * 1485:14bc8e7cc780 tryton/tryton/gui/window/view_form/view/form_gtk/textbox.py: Improve speed of textbox with gtkspell12:30
CIA-5http://hg.tryton.org/tryton/rev/14bc8e7cc78012:30
CIA-5C?dric Krier <ced@b2ck.com> default * 2130:f1bfdb67faaa trytond/ (5 files in 5 dirs): Add comment on table and field for postgresql backend12:32
CIA-5http://hg.tryton.org/trytond/rev/f1bfdb67faaa12:32
CIA-5C?dric Krier <ced@b2ck.com> default * 1486:aeddd0e04b60 tryton/tryton/gui/window/view_form/view/form_gtk/parser.py:12:35
CIA-5Handle always states on group for issue136612:35
CIA-5Allow to override states on label linked to field for issue136712:35
CIA-5http://hg.tryton.org/tryton/rev/aeddd0e04b6012:35
CIA-5ced roundup * #1367/<label> does not honor 'states': [resolved] Fix with changeset aeddd0e04b6012:35
CIA-5http://bugs.tryton.org/roundup/issue136712:35
CIA-5ced roundup * #1366/<group> does not accept attribut 'states': [resolved] Fix with changeset aeddd0e04b6012:35
CIA-5http://bugs.tryton.org/roundup/issue136612:35
-!- LucaSub1(n=LucaSub@host240-173-dynamic.14-87-r.retail.telecomitalia.it) has joined #tryton12:41
-!- essich(n=essich@p4FCF98EA.dip0.t-ipconnect.de) has joined #tryton13:44
-!- cedk(n=ced@gentoo/developer/cedk) has joined #tryton14:00
-!- cedric_b(n=cedric@ANantes-158-1-159-244.w90-59.abo.wanadoo.fr) has joined #tryton14:51
CIA-5paepke roundup * #1350/Error msg on start without python or oo.org:15:44
CIA-5there is a solution on that site:15:44
CIA-5http://www.py2exe.org/index.cgi/Tutorial#Step515:44
CIA-5http://bugs.tryton.org/roundup/issue135015:44
-!- sharoon(n=sharoont@89.243.21.157) has joined #tryton16:07
CIA-5ced roundup * #1350/Error msg on start without python or oo.org:16:27
CIA-5[testing] Could you try without installing Python nor Ooo but with16:27
CIA-5http://www.microsoft.com/downloads/details.aspx?FamilyID=9b2da534-3e03-4391-8a ...16:27
CIA-5http://bugs.tryton.org/roundup/issue135016:27
-!- udono(n=udono@dynamic-unidsl-85-197-19-135.westend.de) has joined #tryton16:28
CIA-5paepke roundup * #1350/Error msg on start without python or oo.org:16:32
CIA-5yes, that works. sorry i didn't mention i have tested that solution.16:32
CIA-5it should be integrated somehow in the installer or at least at the wiki pag ...16:32
CIA-5http://bugs.tryton.org/roundup/issue135016:32
CIA-5ced roundup * #1350/Error msg on start without python or oo.org:16:35
CIA-5Could you test with this one:16:35
CIA-5http://www.b2ck.com/~ced/tryton-setup-1.5.0-20100111.exe16:35
CIA-5http://bugs.tryton.org/roundup/issue135016:35
-!- sharoon(n=sharoont@89.243.21.157) has joined #tryton16:36
-!- juanfer_(n=juanfer@190.27.146.29) has joined #tryton17:19
-!- sharoon(n=sharoont@89.243.21.157) has left #tryton17:41
-!- enlightx(n=enlightx@host-84-220-92-50.cust-adsl.tiscali.it) has joined #tryton19:09
-!- vengfulsquirrel(n=ian@c-69-181-194-95.hsd1.ca.comcast.net) has joined #tryton19:11
-!- LucaSub1(n=LucaSub@host153-175-dynamic.14-87-r.retail.telecomitalia.it) has joined #tryton20:35
cedkvengfulsquirrel: what is the query that takes 5-6 seconds?21:07
vengfulsquirrelhttp://paste.pocoo.org/show/164419/21:42
vengfulsquirrelIm just printing it from execute in database.py for postgresql.21:43
-!- paepke(n=paepke@p54A2A4A2.dip0.t-ipconnect.de) has joined #tryton21:43
vengfulsquirrelthe query is not valid though do you know what happens to en_us ?21:45
vengfulsquirrelcedk:*21:49
cedkvengfulsquirrel: do you have an index on column inventory_line.inventory ?22:19
vengfulsquirrelYes, but I was thinking maybe I need one on inventory_line.product.22:19
vengfulsquirrel"inventory_line_inventory_index" btree (inventory)22:20
cedkvengfulsquirrel: could give the ouput of an explain of the query22:21
cedkvengfulsquirrel: I can not run it because I don't have the right tables22:21
vengfulsquirrelhttp://paste.pocoo.org/show/164451/22:21
cedkvengfulsquirrel: yes I think an index on inventory_line.product will improve the situation22:23
vengfulsquirrelOh right sorry, I already added it before I ran explain so that is with that index, do you think it needs to be a composite index?  Can you tell from the explain output?  It looks like gibberish to me.22:25
vengfulsquirrel                                                         ->  Hash Left Join  (cost=963.30..3516.01 rows=2137 width=128)22:26
vengfulsquirrel                                                               Hash Cond: ("outer".product = "inner".id)22:26
vengfulsquirrelThat is the join on the product right ?22:26
cedkvengfulsquirrel: because of this line: "Seq Scan on product_product ..."22:26
cedkvengfulsquirrel: did you see how the reduce_ids function make a better use of index22:39
vengfulsquirrelIt converts a large IN into something like this right ? http://paste.pocoo.org/show/164461/22:40
cedkvengfulsquirrel: yes and instead of being a test id by id, it can use the index for a range of id22:42
vengfulsquirrelYeah that's a good idea, is it faster in all cases?  Like is there any sort of trade off between the length of the IN and the number of AND conditions ?22:44
cedkvengfulsquirrel: yes, I set the threshold to 5 elements22:47
cedkvengfulsquirrel: but it is en empiric value :-)22:48
-!- tekknokrat(n=gthieleb@dslb-088-075-227-054.pools.arcor-ip.net) has joined #tryton22:54
vengfulsquirrelHa yeah sounds good23:00
yangoon  cedk setup.py of trytond still points to openoffice.interact, but openoffice-python is on pypi23:08
CIA-5C?dric Krier <ced@b2ck.com> default * 2131:b8e52e4d3be1 trytond/setup.py: Rename openoffice.interact into openoffice-python23:19
CIA-5http://hg.tryton.org/trytond/rev/b8e52e4d3be123:19
-!- tekknokrat(n=gthieleb@dslb-088-075-227-054.pools.arcor-ip.net) has left #tryton23:39

Generated by irclog2html.py 2.11.0 by Marius Gedminas - find it at mg.pov.lt!