Skip to content

Commit

Permalink
Merge pull request #1212 from huwjenkins/ctffind_runner-better-error
Browse files Browse the repository at this point in the history
Raise an error when CTF estimation failed on all images; also add LC_ALL=C to avoid problems in CTFFIND. Fixes #1167. Thanks to @huwjenkins.
  • Loading branch information
biochem-fan authored Nov 29, 2024
2 parents 05c903d + db27b77 commit 745ac29
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/ctffind_runner.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -463,6 +463,8 @@ void CtffindRunner::joinCtffindResults()

if (verb > 0 && imic % 60 == 0) progress_bar(imic);
}
if (MDctf.isEmpty())
REPORT_ERROR( (std::string) fn_ctffind_exe + " failed to estimate CTF parameters for any micrograph, exiting...");

if (is_tomo)
{
Expand Down Expand Up @@ -713,7 +715,7 @@ void CtffindRunner::executeCtffind4(long int imic)

// Write script to run ctffind
fh << "#!/usr/bin/env " << fn_shell << std::endl;
fh << fn_ctffind_exe << ctffind4_options << " > " << fn_log << " << EOF"<<std::endl;
fh << "env LC_ALL=C " << fn_ctffind_exe << ctffind4_options << " > " << fn_log << " << EOF"<<std::endl;
// line 1: input image
if (do_movie_thon_rings)
{
Expand Down

0 comments on commit 745ac29

Please # to comment.