[Rapp-dev] Padding for binary morphing: (SIZE + 1) / 2 or SIZE / 2?

[Hans-Peter Nilsson]

In commit
<http://git.savannah.gnu.org/cgit/rapp.git/commit/?id=5c545b7c8b476f2ee7984d51e5648d1a2a1d9f2d>
I fixed what was seen as an inconsistency in the documentation.
I haven't regenerated the one at
<http://www.nongnu.org/rapp/doc/rapp/group__grp__binmorph.html>
yet.  The documentation there defines the radius as the width
rounded up, yet the padding is SIZE / 2, where SIZE is the
maximum size of the structuring element in the horizontal and
vertical directions in width and height of the structural
element, i.e. the "truncated" radius.

It has been argued by local users that the padding should indeed
be SIZE / 2, not (SIZE + 1) / 2, as the former is the
theoretical lower limit.  I.e. the truncated value is correct.
Thus, that the documentation should be changed back (and
clarified), and all overlap-detections correspondingly adjusted.

I'm slightly inclined to do the suggested change, as the code
seems to use the "lower" padding.  Beware, I'm frequently
mistaken, please double-check. ;-)

Since there's no release just yet, this is the best time to
speak up.  Is there any win for the "upper" padding, now or some
conceivable future improvement?

Speaking of the release, I hope to get to that as soon as this
issue is resolved.  I'm also going to add a few pointers to the
project home in the documentation and README, and a rapp.spec
file.

brgds, H-P