search

DerThorsten / nifty

A nifty library for graph based image segmentation.
MIT License
41 stars 21 forks source link

Add test for affinity features #144

Closed constantinpape closed 1 week ago

constantinpape commented 2 years ago

This is pending https://github.com/DerThorsten/nifty/pull/144. (And I don't have time to look into this right now and am a bit puzzled by this issue.)

DerThorsten commented 2 years ago

@constantinpape if you can update me what the issue is I can have a look. Or is the reason for the test the failure I see in the CI? Ie:

UserRangeHistogram::update(): setMinMax(...) has not been called.
constantinpape commented 2 years ago

Hey @DerThorsten it would be great if you could have a look!

Indeed, the issue is the setMinMax() failure in the CI, which only happens on MAC. And now apparently also that the function does not terminate any more on windows. (This used to work, but now seems broken due to my attempts at fixing the issue). On Linux everything works as expected.

This issue happens in accumulateAffinitiesWithAccChain