[Octave-patch-tracker] [patch #9985] [octave forge] (statistics) Add function confusionchart

[Philip Nienhuis]

Follow-up Comment #21, patch #9985 (project octave):

Point taken, yeah I'm sorry that it leads to some extra work for you.

As I wrote elsewhere it's better to cater for dev Octave already now as within
a year (I hope) it´ll be released. Having to rush many updates in many OF
packages at Octave release time when developers (some of which happen to be
package maintainers as well) are usually overloaded anyway is a bad idea.
That said, I do check with current and past releases too, but I care less for
past than for future releases.

Dev Octave is stricter than current 6.2.0, but the consequences haven't
trickled down through the entire code base yet; well, that's what development
branches are for.
I'd rather call it coincidence that your code exposed some of these dev Octave
bugs :-)  My apologies that I sometimes misinterpret newly exposed core Octave
bugs as deficiencies in contributed code. The explanation is that I know
little about classdef and this specific branch of statistics so I can only
concentrate on maintainability (style counts there) and user experience
(adequate docs and input validation). Checking code correctness for me is
mostly limited to checking w. matlab - I have access to some toolboxes.

BTW my patch for the char matrix truncation warning bug has already been
accepted and pushed, so no worries.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/patch/?9985>

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