I have been reviewing the code and looks fine for me too. Some comments:

In 1112: Yes, in ENG will be right, but I think that it is only some kind of 
var control to know when the fixed_costs calculations are deleted.

Good work, when comments are deleted just post here an I will merge it. Thanks!
-- 
https://code.launchpad.net/~gkliska/avanzosc/mrp-6.1_kgb_fix/+merge/176377
Your team Avanzosc Developers is subscribed to branch lp:avanzosc/mrp-6.1.

-- 
Mailing list: https://launchpad.net/~avanzosc
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~avanzosc
More help   : https://help.launchpad.net/ListHelp

Reply via email to