You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by Amol Deshmukh <am...@yahoo.com> on 2005/06/03 14:30:53 UTC

RE: [PATCH] FormulaEvaluator functions added

> ------- Additional Comments From avik@apache.org 
2005-06-01 
> 19:52 -------
> Some testcases failed!
<snipped/>
> 
> Not sure, but these seem to be places where you have
=NA() as 
> the expected
> value. Can you pls investigate, thanks!
> 


Avik,

I presume you opened the test xls in OpenOffice.
The =NA() values are actually Error values 
(#VALUE! etc.), since those tests check if the
evaluator can return error values when that is the
expectation. However, OpenOffice seems to evaluate
some of the formulas that return error in excel.

I have fixed problems in the core eval classes that
was causing the automated tests to fail and updated
the patch - so the new patch file should be good to
go :)

Regards,
~ amol



---------------------------------------------------------------------
To unsubscribe, e-mail: poi-dev-unsubscribe@jakarta.apache.org
Mailing List:    http://jakarta.apache.org/site/mail2.html#poi
The Apache Jakarta POI Project: http://jakarta.apache.org/poi/


RE: [PATCH] FormulaEvaluator functions added

Posted by Avik Sengupta <av...@itellix.com>.
Ok, I'll put it in. Dont assign bugs to yourself, leave it assigned to
be the lists, else we cant figure when you put in new comments :)

On Fri, 2005-06-03 at 05:30 -0700, Amol Deshmukh wrote:
> > ------- Additional Comments From avik@apache.org 
> 2005-06-01 
> > 19:52 -------
> > Some testcases failed!
> <snipped/>
> > 
> > Not sure, but these seem to be places where you have
> =NA() as 
> > the expected
> > value. Can you pls investigate, thanks!
> > 
> 
> 
> Avik,
> 
> I presume you opened the test xls in OpenOffice.
> The =NA() values are actually Error values 
> (#VALUE! etc.), since those tests check if the
> evaluator can return error values when that is the
> expectation. However, OpenOffice seems to evaluate
> some of the formulas that return error in excel.
> 
> I have fixed problems in the core eval classes that
> was causing the automated tests to fail and updated
> the patch - so the new patch file should be good to
> go :)
> 
> Regards,
> ~ amol
> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: poi-dev-unsubscribe@jakarta.apache.org
> Mailing List:    http://jakarta.apache.org/site/mail2.html#poi
> The Apache Jakarta POI Project: http://jakarta.apache.org/poi/
> 
-- 


---------------------------------------------------------------------
To unsubscribe, e-mail: poi-dev-unsubscribe@jakarta.apache.org
Mailing List:    http://jakarta.apache.org/site/mail2.html#poi
The Apache Jakarta POI Project: http://jakarta.apache.org/poi/