commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 20844] - [math] alternative root finding framework and Brent-Dekker solver
Date Tue, 17 Jun 2003 21:22:22 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20844>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20844

[math] alternative root finding framework and Brent-Dekker solver

tobrien@discursive.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED



------- Additional Comments From tobrien@discursive.com  2003-06-17 21:22 -------
Because there was no overlap with the existing code, I've applied these patches 
to the codebase as I think it will make the debate easier to manage.  If we 
find this framework unacceptable we can easily rollback.

I've got a few issues immediately:

1. It didn't compile as submitted. BrentSolver referenced xtol - THIS IS 
IMPORTANT - you need to go in and fix BrentSolver (line 109) it used to 
reference a missing variable xtol, I've changed this to tolerance which is most 
likely wrong.  Also, there were missing parantheses.

2. MathException - I'm not so sure, but I am for throwing a checked exception 
during root finding.  I just don't want to create an exception hierarchy just 
because we can.

3. The tests as submitted do not run successfully.  This may be relates to #1.

Essentially, I view this commit as preliminary, let the discussion begin.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message