Skip to content

Add ability to ignore all errors on run#105

Merged
gnodar01 merged 2 commits intomasterfrom
issues/4527
Jul 28, 2022
Merged

Add ability to ignore all errors on run#105
gnodar01 merged 2 commits intomasterfrom
issues/4527

Conversation

@bethac07
Copy link
Member

In CellProfiler/CellProfiler#4486 and CellProfiler/CellProfiler#4527, as well as at other times, people have asked for ways to just skip one or two bad images. For the reasons laid out in the new ALWAYS_CONTINUE_HELP, that is going to sometimes be a bad idea because it will just kick the can of failure down the road to make you fail later, but for some workflows it's undeniably helpful. Note that this PR only introduces the settings, the companion PR in the main repo will be the only way to actually set them.

ALWAYS_CONTINUE_HELP = """\
Determines whether CellProfiler should always skip images that 
cause errors to be raised vs stopping analysis (in headless mode)
or asking the user what to do (in GUI mode). This may help in 
processing large image sets with a small number of unusual or corrupted
images, but note that the skip of the original image set in question 
may cause issues with modules that look atmultiple image sets across 
groups (CorrectIlluminationCalculate, TrackObjects) or in Export modules,
leading to simply a later failure. Use at your own risk.
"""

@gnodar01 gnodar01 self-requested a review July 28, 2022 15:35
@gnodar01 gnodar01 merged commit ad3eb57 into master Jul 28, 2022
@gnodar01 gnodar01 deleted the issues/4527 branch July 28, 2022 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants