haltSPP_ScreenNormalized check not working correctly

[BeheadedKamikaze]

It appears that in the comparison in the first halt case (line 1356), the variables for samples completed (spp_ScreenNormalized) and the user setting (haltSPP_ScreenNormalized ) have been swapped, resulting in the render terminating as soon as the eye samples have completed (assuming the eye samples complete before the light samples do).

LuxCore/src/slg/film/film.cpp

Line 1356 in b882241

(haltSPP_ScreenNormalized > 0) && (haltSPP_ScreenNormalized > spp_ScreenNormalized)

I think it should appear like the check in the third case (line 1366):

LuxCore/src/slg/film/film.cpp

Line 1366 in b882241

(haltSPP_ScreenNormalized > 0) && (spp_ScreenNormalized > haltSPP_ScreenNormalized)

Potential bug: should the user's CPU be much faster than their GPU, spp_ScreenNormalized exceeds the user setting before spp_PixelNormalized does, then the render might never complete.

[Dade916]

Thanks, I should have fixed the problem.

[BeheadedKamikaze]

Thank-you, Dade!