[Octave-bug-tracker] [bug #65238] Improve nchoosek.m algorithm to prevent numerical issues

[Rik]

Follow-up Comment #4, bug#65238 (group octave):

I made a few more changes in this changeset:
https://hg.savannah.gnu.org/hgweb/octave/rev/39b341c858a7.

1) Use try/catch block to catch mixed integer inputs and emit a meaningul
error message.

2) Replace CamelCase variable "Mx" with "imax" (integer max) for clarity.

3) Add BIST tests for mixed integer inputs and mixed integer / floating point
and mixed single/double inputs.

4) Added note to NEWS file about improvements in speed and precision in
nchoosek.

This function looks a lot better now.  Thank you Henrik.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?65238>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/