[jira] [Work logged] (NUMBERS-131) Re-designing BigFraction.from(double, double, int, int)

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/NUMBERS-131?focusedWorklogId=279385&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-279385 ]

ASF GitHub Bot logged work on NUMBERS-131:
------------------------------------------

                Author: ASF GitHub Bot
            Created on: 18/Jul/19 23:00
            Start Date: 18/Jul/19 23:00
    Worklog Time Spent: 10m 
      Work Description: Schamschi commented on pull request #66: [NUMBERS-131] Re-write implementations of double approximation factory methods in BigFraction
URL: https://github.com/apache/commons-numbers/pull/66
 
 
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Issue Time Tracking
-------------------

            Worklog Id:     (was: 279385)
            Time Spent: 10m
    Remaining Estimate: 0h

> Re-designing BigFraction.from(double, double, int, int)
> -------------------------------------------------------
>
>                 Key: NUMBERS-131
>                 URL: https://issues.apache.org/jira/browse/NUMBERS-131
>             Project: Commons Numbers
>          Issue Type: Improvement
>          Components: fraction
>    Affects Versions: 1.0
>            Reporter: Heinrich Bohne
>            Priority: Minor
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> The method {{BigFraction.from(double, double, int, int)}} can be improved in several ways:
> * It only allows a maximum denominator in the {{int}} range, which defies the purpose of having a {{BigFraction}} class in addition to the class {{Fraction}}. Since {{BigFraction}} is {{BigInteger}} based, it would only be natural to allow the maximum denominator to be passed as a {{BigInteger}}.
> * It only calculates the convergents of the simple continued fraction, but not its semi-convergents, so it doesn't necessarily produce the closest possible approximation within the given bounds.
> * The design is awkward. Making the method's behavior dependent on the values of its arguments is confusing, even the documentation acknowledges this.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)