[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://bz.apache.org/bugzilla/show_bug.cgi?id=54469 Dominik Stadler changed: What|Removed |Added Resolution|--- |LATER Status|NEEDINFO|RESOLVED --- Comment #19 from Dominik Stadler --- 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. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://bz.apache.org/bugzilla/show_bug.cgi?id=54469 Dominik Stadler dominik.stad...@gmx.at changed: What|Removed |Added Keywords||PatchAvailable -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://bz.apache.org/bugzilla/show_bug.cgi?id=54469 Dominik Stadler dominik.stad...@gmx.at changed: What|Removed |Added Component|POI Overall |SS Common -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 Dominik Stadler dominik.stad...@gmx.at changed: What|Removed |Added Status|NEW |NEEDINFO --- Comment #18 from Dominik Stadler dominik.stad...@gmx.at --- needs info based on previous comment -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 Yegor Kozlov ye...@dinom.ru changed: What|Removed |Added Status|REOPENED|NEW --- Comment #17 from Yegor Kozlov ye...@dinom.ru --- 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 -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 Mike Argyriou mich...@gmail.com changed: What|Removed |Added Status|NEEDINFO|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 Mike Argyriou mich...@gmail.com changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED |--- -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 Mike Argyriou mich...@gmail.com changed: What|Removed |Added Attachment #29887|0 |1 is obsolete|| --- Comment #14 from Mike Argyriou mich...@gmail.com --- Created attachment 29986 -- https://issues.apache.org/bugzilla/attachment.cgi?id=29986action=edit updated patch Updated patchsome 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... -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 --- Comment #15 from Mike Argyriou mich...@gmail.com --- Created attachment 29987 -- https://issues.apache.org/bugzilla/attachment.cgi?id=29987action=edit 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 -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 --- Comment #16 from Mike Argyriou mich...@gmail.com --- 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 -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 --- Comment #11 from Mike Argyriou mich...@gmail.com --- 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! -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 --- Comment #12 from Nick Burch apa...@gagravarr.org --- 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 -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 --- Comment #13 from Mike Argyriou mich...@gmail.com --- Thank you for your immediate answer! I will take care of it asap... -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 --- Comment #8 from Nick Burch apa...@gagravarr.org --- Created attachment 29949 -- https://issues.apache.org/bugzilla/attachment.cgi?id=29949action=edit Updated FormulaEvalTestData -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 Nick Burch apa...@gagravarr.org changed: What|Removed |Added Status|REOPENED|NEEDINFO --- Comment #9 from Nick Burch apa...@gagravarr.org --- 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? -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 --- Comment #10 from Mike Argyriou mich...@gmail.com --- Can you attach the unit test? -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 --- Comment #7 from Mike Argyriou mich...@gmail.com --- Is there any feedback for this patch? -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 Mike Argyriou mich...@gmail.com changed: What|Removed |Added Status|REOPENED|NEEDINFO Severity|normal |enhancement -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 Mike Argyriou mich...@gmail.com changed: What|Removed |Added Status|NEEDINFO|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 Mike Argyriou mich...@gmail.com changed: What|Removed |Added Status|NEEDINFO|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 Mike Argyriou mich...@gmail.com changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED |--- -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 Mike Argyriou mich...@gmail.com changed: What|Removed |Added Version|3.8-dev |4.0-dev Summary|IPMT, PPMT financial|[PATCH] IPMT, PPMT |functions implemented and |financial functions |integrated in POI v3.8 |implemented and integrated ||in latest POI --- Comment #5 from Mike Argyriou mich...@gmail.com --- 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. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
[Bug 54469] [PATCH] IPMT, PPMT financial functions implemented and integrated in latest POI
https://issues.apache.org/bugzilla/show_bug.cgi?id=54469 --- Comment #6 from Mike Argyriou mich...@gmail.com --- Created attachment 29887 -- https://issues.apache.org/bugzilla/attachment.cgi?id=29887action=edit diff patch file created from ant -f patch.xml on the latest svn POI repository -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, e-mail: dev-h...@poi.apache.org
