Difference in behavior when num_polar_sections != num_azimuthal_sections

[matthewturk]

I'm working through things related to #203 and #204 and I believe I have identified the principle problem, which is that there is some behavioral difference in the initialization of center_to_polar_bound_vectors_ and center_to_azimuthal_bound_vectors_ when num_polar_sections == num_azimuthal_sections and when the converse. The more correct (appearing) behavior is when they are equal, but I'm not sure I can completely convince myself of the correctness of that. The thing that gives me the most pause is that in the case that the count is equal, I don't see why it would get the correct answer for the values for both polar and azimuthal, since they span different values.

Anyway, I'm working on it, and have a partially completed pull request in the works.

[ak-2485]

"The thing that gives me the most pause is that in the case that the count is equal, I don't see why it would get the correct answer for the values for both polar and azimuthal, since they span different values." By spanning different values are you talking about the range restriction on the angle of inclination/azimuthal angle?

[matthewturk]

Yup!

…

On Thu, Aug 6, 2020 at 9:06 PM Ariel Kellison ***@***.***> wrote: "The thing that gives me the most pause is that in the case that the count is equal, I don't see why it would get the correct answer for the values for both polar and azimuthal, since they span different values." By spanning different values are you talking about the range restriction on the angle of inclination/azimuthal angle? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#205 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAVXO3VKOND3XX3JEPWVCLR7NOT7ANCNFSM4PUYF5MA> .

[cgyurgyik]

I've reduced this to only use a single set of calculations, rather than optimize for the case when num_angular == num_azimuthal. PTAL at #208 , paying special attention to start bounds, end bounds, and radii used for calculations. I believe the addition of MaxBound and MinBound may lead to some slight miscalculations in some places.