IRC logs of #tryton for Thursday, 2010-08-05

chat.freenode.net #tryton log beginning Thu Aug 5 00:00:01 CEST 2010
-!- sharoon(~sharoon@opg066b.halls.manchester.ac.uk) has joined #tryton01:11
-!- digitalsatori(~tony@116.233.249.147) has joined #tryton02:03
-!- ikks(~ikks@190.158.116.213) has joined #tryton03:55
-!- juanfer(~juanfer@190.27.84.91) has joined #tryton04:32
-!- gremly(~gremly@201.244.193.238) has joined #tryton04:54
-!- mr_amit(~amit@117.254.177.188) has joined #tryton06:40
-!- Milyardo(~zpowers@c-98-209-86-19.hsd1.mi.comcast.net) has joined #tryton06:46
-!- digitalsatori(~tony@116.233.249.147) has joined #tryton06:47
-!- Milyardo(~zpowers@c-98-209-86-19.hsd1.mi.comcast.net) has joined #tryton07:09
-!- tekknokrat(~lila@dslb-188-102-055-145.pools.arcor-ip.net) has joined #tryton07:38
-!- cedk(~ced@gentoo/developer/cedk) has joined #tryton07:45
-!- pheller(~pheller@pool-173-48-216-195.bstnma.fios.verizon.net) has joined #tryton07:46
-!- Timitos(~timitos@88.217.184.172) has joined #tryton07:47
phellercedk: you there?08:16
phellerI'd like to fix this last thing in sale before I go to sleep.08:16
phellerbut I'm not quite sure what is wrong here:  http://codereview.appspot.com/1892048/diff/9001/10002#newcode33408:17
cedkpheller: there was a context created with get_tax_context08:42
cedkpheller: but it was not used08:42
cedkpheller: in fact it must be used for the call tax_obj.compute08:43
cedkACTION bbl08:43
-!- digitalsatori(~tony@116.233.249.147) has joined #tryton08:43
-!- eLBati(~elbati@94.163.67.33) has joined #tryton08:50
-!- cedk(~ced@gentoo/developer/cedk) has joined #tryton09:08
-!- Milyardo(~zpowers@c-98-209-86-19.hsd1.mi.comcast.net) has joined #tryton09:14
-!- johbo(~joh@statdsl-085-016-072-173.ewe-ip-backbone.de) has joined #tryton09:58
-!- ecarreras(~under@unaffiliated/ecarreras) has joined #tryton10:15
cedkecarreras: hi10:56
ecarrerashi cedk :)10:56
ecarrerasI'm definitely moving to tryton :)10:58
ecarrerasnext week I've holidays, I'll spent some time to study migration process10:59
-!- FWiesing(~FWiesing@85-126-100-130.work.xdsl-line.inode.at) has joined #tryton11:05
cedkecarreras: do you know your requirements?11:09
ecarreraswell I've to move openerp modules code to tryton code11:11
cedkecarreras: which ones?11:12
ecarrerasare client specific11:12
cedkecarreras: but which fonctionnality do you use from OE? To see if you will have all of them in Tryton11:13
ecarrerasthe backend of database, and easy development11:13
ecarrerasI've been using OpenERP as a framework11:14
ecarrerasto develop a platform on top11:14
ecarrerasI think all functionalities that I use in OE are in Tryton too :)11:15
cedkecarreras: yes if you don't use any module11:16
-!- blast_hardcheese(~blast_har@dsl092-043-124.lax1.dsl.speakeasy.net) has joined #tryton11:16
-!- bechamel(~user@host-85-201-159-186.brutele.be) has joined #tryton11:17
ecarrerasI'm using base for partners that's all11:17
cedkecarreras: you will even have more functionalities like historization, WebDAV, PYSON etc.11:17
ecarrerasyeah and a lot of bugfixes done :)11:17
ecarrerasfirst day in the fork in tryton, the remote code execution in pickle were solved11:18
ecarreras(via NET-RPC)11:18
ecarreras(now) OpenERP gtkclient have remote code execution when connecting to a malicious ERP server with net-rpc protocol11:19
ecarrerasand decimal problem...11:20
-!- paepke(~paepke@p4FEB006C.dip0.t-ipconnect.de) has joined #tryton11:21
ecarrerasand the last... the date widget... How can they say that the date is invalid?? lol 1850-12-23 seems valid for me...11:21
ecarrerasanyway, I'll make the change :D11:22
cedkecarreras: it comes from Python datetime strptime method11:23
cedkecarreras: they have limitation on some OS (like Window$)11:23
cedkecarreras: in fact, you will see that when OE addes new features they do it fast but don't go in deep11:25
ecarrerascedk, totally agree11:26
ecarrerassecurity in openerp doesn't seems important11:26
cedkecarreras: not only security but the all software11:27
ecarrerasthey don't have codereview11:27
ecarrerasand they aren't proactives to accept patches, they only want that OPENERP S.A. have the all copyrighted code11:28
ecarreraslike changing the license to Affero without asking11:29
ecarrerasI'm tired to fight in launchpad with some workers of OpenERP S.A. which don't know the code of OpenERP in deep11:31
ecarrerasanyway, I'm totally convided to migrate to tryton :)11:32
ecarrerass/convided/convinced/g11:32
-!- eLBati(~elbati@94.162.29.80) has joined #tryton12:05
-!- Milyardo(~zpowers@c-98-209-86-19.hsd1.mi.comcast.net) has joined #tryton13:27
cedkecarreras: I guess no need to tell you that if you have issues to migrate on Tryton, you can ask on irc or tryton-dev mailing list13:39
ecarrerasthanks cedk :)13:39
-!- woakas(~woakas@pcsp163-59.supercabletv.net.co) has joined #tryton14:06
-!- mr_amit(~amit@117.254.177.188) has joined #tryton14:08
-!- digitalsatori(~tony@116.233.249.147) has joined #tryton14:24
-!- eLBati(~elbati@93.37.82.47) has joined #tryton15:51
-!- mr_amit(~amit@117.254.17.126) has joined #tryton16:08
-!- zodman(~Miranda@67.223.236.231) has joined #tryton16:09
-!- Red15(~red15@unaffiliated/red15) has joined #tryton16:24
-!- pepeu(~manuel@201.155.193.192) has joined #tryton16:39
-!- gremly(~gremly@201.244.193.238) has joined #tryton16:47
-!- gremly(~gremly@201.244.193.238) has joined #tryton16:49
-!- pheller(~pheller@pool-173-48-216-195.bstnma.fios.verizon.net) has joined #tryton16:52
phellercedk: you there?17:01
cedkpheller: yes17:06
phellercedk: just emailed you a comment on that last issue in Sale17:06
cedkpheller: the context should be used for tax compute17:11
phellerso something like "with Transaction().set_context(self.get_tax_context(vals)):" just before "for tax in tax_obj.compute(...." ?17:14
phellerI'm sorry, I'm a little confused because "ctx", which was the new context, was not passed as an argument in the original version....17:15
cedkpheller: yes it was an error17:17
phellerah ha, so the context argument to tax_obj.compute should have been context=ctx ?17:18
pheller.... but the other functions in that for loop (invoice_obj._compute_tax) -- this should still use the standard context?17:19
cedkpheller: yes17:36
phellerok, changes uploaded to codereview17:47
phellerok, lunch time.  back in an hour17:48
-!- paepke_(~paepke@p4FEB006C.dip0.t-ipconnect.de) has joined #tryton17:53
-!- FWiesing(~FWiesing@85-126-100-130.work.xdsl-line.inode.at) has left #tryton18:05
-!- pheller(~pheller@c1fw226.constantcontact.com) has joined #tryton18:27
-!- cedk(~ced@gentoo/developer/cedk) has joined #tryton18:33
phellercedk: oops!  I thought the multiplication occurred inside parentheses, hence why I removed the line continuation.  I've put it back.18:52
-!- tekknokrat(~lila@dslb-188-102-055-145.pools.arcor-ip.net) has joined #tryton19:16
-!- paepke(~paepke@p4FEB006C.dip0.t-ipconnect.de) has left #tryton19:52
-!- magt1(~Tushar@27.107.111.79) has joined #tryton20:24
-!- magt(~Tushar@27.107.80.94) has joined #tryton20:30
-!- eLBati(~elbati@94.162.55.169) has joined #tryton20:50
-!- paepke(~paepke@p5B32C2CA.dip.t-dialin.net) has joined #tryton22:00
-!- juanfer(~juanfer@190.27.84.91) has joined #tryton23:06
cedkpheller: I don't change code that is not modified by the Transaction switch23:06
phellerah, ok23:07
phellerI won't make those comments on your code then23:07
phellerI have changed a few of the line continuation bits, but that is all....23:08
phellerI'll just stick to fixes of the Transaction() thing now23:08
cedkpheller: it is a matter of quality, changeset must be focus on one thing23:09
phelleryes, I understand.23:09
cedkpheller: those change could be not in an other changeset23:10
phellerok23:11
cedkpheller: but keep the testing :-)23:12
phellerI will.  this is good practice for me.  I've never done much team oriented development.23:13
phellerI wish rietveld integrated with eclipse ;-)23:13
cedkpheller: you can ask guido :-)23:14
phellerMy guess is that guido's development environment is emacs or vim, and nothing else.23:14
cedkpheller: I don't know but I know that he debugs with print statement23:15
phellerouch23:16
cedkpheller: I do the same ;-)23:16
cedkpheller: never used pdb23:16
phellerare you using emacs or vi?23:17
cedkpheller: vim23:17
phellercedk: you should try eclipse+pydev+vwrapper23:18
phellercedk: vwrapper gives you vim style editing in the editor windows, but you can still use the mouse, and commands like copy/cut/paste if you want23:18
cedkpheller: I hate mouse23:18
cedkpheller: I almost never use it23:19
phellercedk: well, it's no problem to make keyboard shortcuts for everything in eclipse too23:19
phellercedk: it's most useful for debugging, though.23:19
cedkpheller: I'm well with print statement23:20
phellercedk: breakpoints, variable watches, stepping through the code visually23:20
phellercedk: no worries, just sharing my experience23:20
cedkpheller: I know debuggers, I used gdb a lot when developping in C23:20
phellerah, ok23:21
cedkpheller: but I find Python doesn't require one because you don't have the same error then in C23:21
phellerthis is true, normally the stack trace is quite informative23:21
cedkpheller: yes, most of the time I fix bug with only traceback23:22
cedkpheller: and I just check if it is well fixed by testing23:22
cedkpheller: if not, then a few print statement and I understand what is happening23:22
phellercedk: an example of when I find it most useful -- finding places where code blocks; like synchronous sockets, etc23:22
cedkpheller: but of course this is because I know well the code23:23
phellercedk: yep.  so you worked at tiny before?23:24
cedkpheller: yes23:25
phellercedk: this explains why you know the code well!23:25
phellercedk: I've noticed in doing some of these conversions that the unittests do not yet have really wide coverage -- is this in the roadmap?23:26
cedkpheller: we have focus effort of unittest on the core first23:27
cedkpheller: the coverage of the core is quiet good23:27
cedkpheller: but still misses some part23:27
phellercedk: makes sense.  I think this is they way to avoid the problems openerp has with calculating things incorrectly....23:28
cedkpheller: but new code now is always tested23:28
phellercedk: and to prevent regressions, naturally23:28
cedkpheller: but we lake of devs23:29
cedkpheller: that is why I tried to start the bugs of the month23:30
cedkI don't know if it will work as I had any feedback until now23:30
phellercedk: yep.  I will try to help with this, after the Transaction() work is finished.23:30
cedkpheller: yes of course23:31
phellercedk: and I hope that the one2one can be in the near future too....23:32
-!- paepke_(~paepke@p5B32DA75.dip.t-dialin.net) has joined #tryton23:32
phellercedk:  it occurs to me that the other thing I mentioned (product attributes, then product instances with values for the attributes) -- is the "entity attribute value" model which was discussed on the openerp-experts-framework list recently23:32
cedkpheller: I don't think23:33
cedkpheller: they talked about added attributes on the product model instead of extending it by adding new columns in the table23:34
phellercedk: I think the concept I am looking for is similar, but maybe not exactly the same23:34
cedkpheller: I think you need some kind of serial number with lists of attributes23:36
phellercedk: yes, but I want to define that list of attributes on the product, and then define the values for these attributes with the serial number23:37
cedkpheller: yes23:37
phellercedk: for instance, one of these attributes would actually be "serial number"23:37
phellerI started working on a module to add all of this, but I decided to wait until the transaction work is done23:37
cedkpheller: yes I talked about serial number because I don't have yet better name for this23:38
phellercedk: "instance" ?23:39
cedkpheller: don't know yet23:39
-!- FWiesing(~FWiesing@85-126-100-130.work.xdsl-line.inode.at) has joined #tryton23:56

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