This vote on rc1 is cancelled. I am going to start on rc2 but it may be a 
couple of days before I have time to complete it.
-Matt

________________________________
From: Matt Juntunen <matt.juntu...@hotmail.com>
Sent: Saturday, July 4, 2020 7:57 AM
To: Commons Developers List <dev@commons.apache.org>
Subject: Re: [VOTE] Release Apache Commons Geometry (full distribution) 
1.0-beta1 based on RC1

Thank you for the thorough review, Xeno! I'm pulling those changes in now.
-Matt
________________________________
From: Xeno Amess <xenoam...@gmail.com>
Sent: Friday, July 3, 2020 11:17 PM
To: Commons Developers List <dev@commons.apache.org>; Matt Juntunen 
<matt.juntu...@hotmail.com>
Subject: Re: [VOTE] Release Apache Commons Geometry (full distribution) 
1.0-beta1 based on RC1

Hi.
I finished the review.
Suggestions details are listed on 
https://github.com/apache/commons-geometry/pulls
Notice that I am not familiar to this repo, nor a math expert, thus you should 
not fully trust my prs, you need to see them, and check if every change 
correct, because sometimes I make mistakes.

Xeno Amess <xenoam...@gmail.com<mailto:xenoam...@gmail.com>> 于2020年7月4日周六 
上午10:09写道:
Hi.
I see in some places of the repo, two double fields are compared using ==.
Shouldn't we use Double.compare?
Or is it by design?

example places:

PolarCoordinates.equals
SphericalCoordinates.equals
Vector1D.angle
Vector2D.equals
Vector3D.equals
Vectors.isRealNonZero

Xeno Amess <xenoam...@gmail.com<mailto:xenoam...@gmail.com>> 于2020年7月4日周六 
上午1:51写道:
Hi.
Please also consider about https://github.com/apache/commons-geometry/pull/85

I will perform more analyze opun this repo when you solve this two prs.

Xeno Amess <xenoam...@gmail.com<mailto:xenoam...@gmail.com>> 于2020年7月4日周六 
上午1:01写道:
Hi.
Please check https://github.com/apache/commons-geometry/pull/84
In short, there be too many errors in javadoc.
(and this is the reason why I made 
https://github.com/apache/commons-parent/pull/7)

There still exist some other problems, but I'd prefer check them after this pr, 
as resolve conflict is boring.


Xeno Amess <xenoam...@gmail.com<mailto:xenoam...@gmail.com>> 于2020年7月4日周六 
上午12:13写道:
Hi.
I see somethings not quite right already.
Will start pr directly.

Xeno Amess <xenoam...@gmail.com<mailto:xenoam...@gmail.com>> 于2020年7月3日周五 
下午4:20写道:
I might review it tonight if I have time...
Only if I have time.
Really busy these days...

Reply via email to