openerp-community team mailing list archive

[ali-mis <ahmadi_mis@xxxxxxxxxxx>]

-------------------------------------------
Regards,
Ali AL-Ahmadi
+966597057057
 


To: mp+127138@xxxxxxxxxxxxxxxxxx; moylop260@xxxxxxxxxx; fp@xxxxxxxxxxx
From: nhomar@xxxxxxxxx
Date: Sun, 30 Sep 2012 05:19:31 +0000
Subject: [Openerp-community] [Merge] lp:~openerp-community/openobject-addons/trunk-btree_compute-vauxoo into lp:openobject-addons

Nhomar Hernandez (Vauxoo) has proposed merging lp:~openerp-community/openobject-addons/trunk-btree_compute-vauxoo into lp:openobject-addons.
 
Requested reviews:
  Olivier Dony (OpenERP) (odo-openerp)
  OpenERP R&D Team (openerp-dev): finish, learn improve
Related bugs:
  Bug #887376 in OpenERP Addons: "[6.0 & 6.1] [account] def compute needs optimization"
  https://bugs.launchpad.net/openobject-addons/+bug/887376
 
For more details, see:
https://code.launchpad.net/~openerp-community/openobject-addons/trunk-btree_compute-vauxoo/+merge/127138
 
Account compute with SQL b-tree
 
Description:
 
With severals accounting entries the algorith to compute totals on accounting is so unefficient, It is happening because we are not using b-tree sql but all data on chart of account is ready to use with this advantage, some information about what b-tree is;:
 
http://www.simple-talk.com/sql/t-sql-programming/binary-trees-in-sql/
 
We have 3 big implementations with +1000 accounts and +2Millions of records, compute the tree view on chart of account was a big problem and the speed on hardware never was enought, for this reason we took the time of rewrite all compute method and prepare this merge proposal.
 
>From Vauxoo we are tottally sure we can not stay +1 year waiting for this improvement due to we are becoming the biggest open source ERP system and community.
 
Please test and give us feedback.
 
Some Important points:
 
We are proposing this MP with both methods and one variable to be able to use both, technically it not so ellegant but due to the impact that accounting can receive with this improvement we are being carefully, you can manage both methods with the same code, just uncomment one line and your code will be the original one.
 
Some comments on:
 
https://code.launchpad.net/~vauxoo/openobject-addons/trunk-bug_887376-account_move_compute_with_btree-moylop260/+merge/89623
 
But these is for work with testing YAML
 
FUNCTIONAL LIST:
  -Calculation field foreign currency
 
YAML TEST:
  -Draft are not considered in the sum.
  -Consolidated accounts.
  -2 Invoices.
  -Multicurrency invoices.
  -Draft and unbalanced entries.
  -Consolidated of different multi-company
  
Please Help us to develop several YAML tests to be sure everything will be fine with it!
 
Resubmited for internal Launchpad Error with diff! - I hope mantain this branch alive until everybody test it.
 
All around this improvement can impact A LOT the generation of Financial reports
-- 
https://code.launchpad.net/~openerp-community/openobject-addons/trunk-btree_compute-vauxoo/+merge/127138
Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-addons/trunk-btree_compute-vauxoo.

_______________________________________________ Mailing list: https://launchpad.net/~openerp-community Post to : openerp-community@xxxxxxxxxxxxxxxxxxx Unsubscribe : https://launchpad.net/~openerp-community More help : https://help.launchpad.net/ListHelp 		 	   		  
-- 
https://code.launchpad.net/~openerp-community/openobject-addons/trunk-btree_compute-vauxoo/+merge/127138
Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-addons/trunk-btree_compute-vauxoo.