[Biojava-dev] [Bug 2298] Use of Big Decimal in DistributionTrainer

[bugzilla-daemon at portal.open-bio.org]

http://bugzilla.open-bio.org/show_bug.cgi?id=2298


holland at ebi.ac.uk changed:

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




------- Comment #1 from holland at ebi.ac.uk  2007-05-18 09:54 EST -------
Do you want double changed to BigDecimal for _every_ occurrence of double (e.g.
weights, scores, etc.) or just for one of these occurrences? 

Modifying the existing classes is going to produce very inefficient code as
there is a lot of use of the public API internally, which would require
boxing/unboxing of double values to/from BigDecimal values. Such changes are
possible but would cause very slow code due to the overhead of doing all these
conversions.

Better quality code could be produced if we just rewrote the distribution code
completely with BigDecimals in mind. Extending the existing modules would not
suffice because the public API is too tightly linked with the internal
calculations, and to add in BigDecimal-awareness would be tantamount to a
complete rewrite anyway!

I think we can leave this until 1.6, but I'll leave it as open and assigned so
that we don't forget it.


-- 
Configure bugmail: http://bugzilla.open-bio.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.