astrorama / SourceXtractorPlusPlus

SourceXtractor++, the next generation SExtractor
https://astrorama.github.io/SourceXtractorPlusPlus/
GNU Lesser General Public License v3.0
72 stars 9 forks source link

SE++ processing fades out #461

Closed mkuemmel closed 2 years ago

mkuemmel commented 2 years ago

Following the discussions this morning I have uploaded the 21GB dataset to here: https://deepdip.iap.fr/#item/62332d6381e19044840fe096

The two shell scripts in there both do not run, which means they fade away after the background modelling. Note that:

Could be related to #445 though some modifications have been done but it does not help in this case.

ayllon commented 2 years ago

Hi Martin,

At some point both model coordinates (x and y) are NaN, which on a debug builds triggers an assert error but on a release build likely keeps going until something else triggers another error that kills the thread.

It may be somehow related to the bug on the iterative model fitting that @marcschefer mentioned yesterday, since when I go back to the "bulk" model fitting with

use_iterative_fitting(False)

It runs just fine.

mkuemmel commented 2 years ago

Thanks for checking that so quick! I can confirm that switching off iterative fitting solves it.

I was wondering by myself whether the iterative model fit bug causes this problem, but in principle all detected objects are on the measurement images.

I just checked, with #462 iterative model fitting can be switched on again and seems to solve the problem.

ayllon commented 2 years ago

I think we can close this one?

mkuemmel commented 2 years ago

Right, we can!