[Octave-bug-tracker] [bug #55172] [octave forge] (image) b = bwmorph (BW, 'thin', 1) is not Matlab compatible

[Hartmut]

Follow-up Comment #3, bug #55172 (project octave):

* I haven't tested the output values of your supposed function, yet. Have you?
Dou you get Matlab compatible results for a couple of meaningful examples? If
yes, could you add tests for those to bwmorph.m? (Do we need to remove
existing tests in bwmorph?)
* Could you add the content of your thin_fun1.m and thin_fun2.m somehow to the
existing private function __conditional_mark_patterns_lut_fun__ ? I like the
way this was done before: the final LUT numbers were hard-coded into the code
of bwmorph.m, but  there was also a (commented out) code line how to produce
this LUT via the __conditional_mark_patterns_lut_fun__ function. This made it
possible for me to understand where those numbers (the LUT) came from, and
imprived the maintainability of the code.
* Could you add the detailed paper reference into the code? (Maybe into
__conditional_mark_patterns_lut_fun__ .m?)

    _______________________________________________________

Reply to this item at:

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

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