Re: [numbers-fraction] Double approximation constructor/factory method overhaul

2019-07-19 Thread Heinrich Bohne
Sure, no problem, thanks for letting me know! On 7/19/19 6:22 PM, Eric Barnhill wrote: I'm looking forward to reviewing your code within my limited knowledge, however I can't guarantee a quick time frame since Apache GSoC mentor milestones are due next week and I think that could get time

Re: [numbers-fraction] Double approximation constructor/factory method overhaul

2019-07-19 Thread Eric Barnhill
I'm looking forward to reviewing your code within my limited knowledge, however I can't guarantee a quick time frame since Apache GSoC mentor milestones are due next week and I think that could get time consuming. Thanks for the contribution, Eric On Thu, Jul 18, 2019 at 4:13 PM Heinrich Bohne

Re: [numbers-fraction] Double approximation constructor/factory method overhaul

2019-07-18 Thread Heinrich Bohne
So, I think the code I have so far is ripe for a pull request, so I submitted one. I changed the contracts of the epsilon and max-denominator factory methods, because the old specifications were not very useful, especially that of the max-denominator method – the only guarantee you could get from

Re: [numbers-fraction] Double approximation constructor/factory method overhaul

2019-07-17 Thread Heinrich Bohne
It just occured to me that you might have misunderstood my sentence: I am even more confused by your suggestion seeing as it was you who banned BigInteger from Fraction.addSub(Fraction, boolean) in https://issues.apache.org/jira/browse/NUMBERS-79  , which, even though you were not aware of it

Re: [numbers-fraction] Double approximation constructor/factory method overhaul

2019-07-17 Thread Heinrich Bohne
The reason it was done was because Knuth proved (as in mathematical proof) that a long is insufficient for certain fraction multiplications where both numerator and denominator are large ints; 65 rather than 64 bits are necessary and a long will not suffice. You seem to have missed my comment

Re: [numbers-fraction] Double approximation constructor/factory method overhaul

2019-07-16 Thread Eric Barnhill
On Tue, Jul 16, 2019 at 2:41 PM Heinrich Bohne wrote: > > Do you think we really even need a BigFraction class at all in the > context > > of these upgrades? Or should one of the Fraction factory methods just > take > > BigInteger argumentsm and all fractions use the lazy dynamic method of > >

Re: [numbers-fraction] Double approximation constructor/factory method overhaul

2019-07-16 Thread Heinrich Bohne
Do you think we really even need a BigFraction class at all in the context of these upgrades? Or should one of the Fraction factory methods just take BigInteger argumentsm and all fractions use the lazy dynamic method of calculation you are proposing? I don't quite understand what you mean by

Re: [numbers-fraction] Double approximation constructor/factory method overhaul

2019-07-16 Thread Eric Barnhill
Sorry for the delay, I was on vacation. On Fri, Jul 5, 2019 at 2:09 PM Heinrich Bohne wrote: > Hello! > > I think a re-design of the factory method BigFraction.from(double, > double, int, int) is in order, because I see several problems with it: > > First, having a separate fraction class

Re: [numbers-fraction] Double approximation constructor/factory method overhaul

2019-07-05 Thread Gilles Sadowski
Hello Heinrich. Le ven. 5 juil. 2019 à 23:09, Heinrich Bohne a écrit : > > Hello! > > I think a re-design of the factory method BigFraction.from(double, > double, int, int) is in order, because I see several problems with it: > > First, having a separate fraction class intended to overcome the >