[ 
https://issues.apache.org/jira/browse/MATH-181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12561796#action_12561796
 ] 

Luc Maisonobe commented on MATH-181:
------------------------------------

I find the javadoc of the private four arguments constructor misleading.
It does not really enforce the fact the result is within epsilon of the true 
result in all cases. The constructor seems designed for only two different call 
sequence: non-zero epsilon and INTEGER.MAX_VALUE maxDenominator on the one 
hand, ignored epsilon (set to 0 in the public constructor) and non-maximal 
maxDenominator on the other hand. The current javadoc seems to say both 
constraints on epsilon and maxDenominator are enforced at the same time, I 
don't think it's true.

> Specify the maximum of digits when parsing a Fraction
> -----------------------------------------------------
>
>                 Key: MATH-181
>                 URL: https://issues.apache.org/jira/browse/MATH-181
>             Project: Commons Math
>          Issue Type: Improvement
>    Affects Versions: 1.1
>            Reporter: Niall Pemberton
>            Priority: Minor
>         Attachments: MATH-181-FractionDigitsLimit.patch
>
>
> Firstly, thanks for the Fraction code - I've adapated it for something I'm 
> working on and I didn't have a clue how to convert a decimal to a fraction :)
> Excel spreadsheets have the facility to specify a fraction format where you 
> specify the maximum number of denominator digits to display.
> So for example:
>     format "?/?" displays decimal values formatted in the range 1/2 to n/9
>     format "??/??" displays decimal values formatted in the range 1/2 to n/99
>     format "???/???" displays decimal values formatted in the range 1/2 to 
> n/999 etc
> In excel then the value 0.6152 displays as 3/5, 8/13 and 510/829 respectively 
> for the above 3 formats.
> I'm attaching a patch for the Fraction class which adds a new constructor 
> where the maximum number of digits can be specified, rather than the epsilon 
> value.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to