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...