I have extended POI to contain the EXCEL fincancial functions IPMT, PPMT. Just diff and integrate the differences... is this the right place for posting the patched POI?
Because the patched POI is too large to attach it here (about 100MB) you can download it from http://uploaded.net/file/ulq8cj21
The patch should normally be just a few kb in size, maybe a few hundred kb with test files. Are you able to check that your patch only has these, and not something else by mistake? (100mb is a pretty huge patch!)
Actually I have uploaded all the POI project patched with the new code. Should I upload only the diffs?
yes, you should upload only the diffs. See http://poi.apache.org/guidelines.html#SubmittingPatches (In reply to comment #3) > Actually I have uploaded all the POI project patched with the new code. > Should I upload only the diffs?
I have downloaded the latest POI from git, integrated/created the IPMT,PPMT functions and created the patched using "ant -f patch.xml". I have attached the path created by Ant.
Created attachment 29887 [details] diff patch file created from "ant -f patch.xml" on the latest svn POI repository
Is there any feedback for this patch?
Created attachment 29949 [details] Updated FormulaEvalTestData
The patch doesn't include any unit tests, so I tried adding one myself by putting in a few sample formulas using the functions in FormulaEvalTestData.xls However, when I do that, the unit test fails as POI is unable to correctly evaluate the PPMT function to give the same result as Excel Could you please look into this, and produce either an updated patch that fixes it, or suggest an alternate test for it?
Can you attach the unit test?
Nick can you attach the unit tests please so I can check why they fail in order the patch to be integrated to POI? Thank u!
If you grab the POI sourcecode, you'll find the unit test in there already. If you replace FormulaEvalTestData in test-documents with the one attached, and re-run the test with your patch applied you'll see the failure
Thank you for your immediate answer! I will take care of it asap...
Created attachment 29986 [details] updated patch Updated patch....some minor fixes and some typos... The tests should pass if you set the formatting of the cells in FormulaEvalTestData.xls to more decimal digits... the IPMT test fails because the cell D764 with value 0.01 has two decimals digits...the test returns the value ~0.005 and it fails..increase the cell's number of decimal digits to (e.g.) 4...
Created attachment 29987 [details] Main classes for testing ipmt, pmt, etc The attachnent contains Main-classes which test the financial functions...check the code please... place all the *.java in trunk/src/java
I have uploaded an updated patch and new convinient manual tests (should not be included in the POI trunk repository).. Just read the attachments comments..in the nutshell please increase the decimal digitis on the test-xls. Thanks you Mike
Partially applied in r1451886 with some tweaks. Your implementation of PPMT is buggy. If I run PPMT_test I'm getting this: ppmt=-121.95920277082378 [should be equal to -350.99] The value -350.99 is correct, I confirmed it in Excel. Interestingly that two test cases from http://office.microsoft.com/en-001/excel-help/ppmt-function-HP010342774.aspx pass. Please fix PPMT , I guess it a math error and should be easy to detect and fix. Regards, Yegor
needs info based on previous comment
No update received for a long time thus I am closing this, please report an refreshed/fixed patch for current trunk if this is still of interest to you.