aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorThomas White <taw@physics.org>2022-03-03 12:33:05 +0100
committerThomas White <taw@physics.org>2022-03-03 12:33:05 +0100
commit634ac34f8ffde1d3991751230c57eba70529f005 (patch)
tree58dbf7dd27c23a2d9d1653358f78980eecb24ebb /src
parent049d7fb351bfaf4ae2e627fe21f8748504a42778 (diff)
GUI: Better handling of error-filled indexamajig logs
If there are lots of error messages (e.g. a missing mask file), looking at the last 4k of the log file is not always enough. Now it will go back to 16k, in steps of 4k. It will stop early if it finds the start of the file, and it will complain if it still doesn't find what it's looking for.
Diffstat (limited to 'src')
-rw-r--r--src/gui_index.c55
1 files changed, 39 insertions, 16 deletions
diff --git a/src/gui_index.c b/src/gui_index.c
index b5b24248..99d8feb3 100644
--- a/src/gui_index.c
+++ b/src/gui_index.c
@@ -1015,30 +1015,53 @@ int read_number_processed(const char *filename)
{
FILE *fh = fopen(filename, "r");
int n_proc = 0;
+ long len = 0;
+ int found;
/* Normal situation if SLURM job hasn't started yet */
if ( fh == NULL ) return 0;
- /* Only look at the last part of the file */
- fseek(fh, -4096, SEEK_END);
-
do {
- char line[1024];
- if ( fgets(line, 1024, fh) == NULL ) break;
- if ( strncmp(line, "Final: ", 7) == 0 ) {
- int i;
- if ( sscanf(line, "Final: %i images processed", &i) == 1 ) {
- n_proc = i;
- }
- } else if ( strstr(line, " images processed, ") != NULL ) {
- int i;
- if ( sscanf(line, "%i ", &i) == 1 ) {
- n_proc = i;
- }
+ len += 4096;
+
+ /* Only look at the last part of the file */
+ if ( fseek(fh, -len, SEEK_END) ) {
+ /* Whoops, tried to go too far.
+ * Start from beginning, and don't loop again */
+ fseek(fh, 0, SEEK_SET);
+ found = 1;
+ } else {
+ found = 0;
}
- } while ( 1 );
+ do {
+ char line[1024];
+ if ( fgets(line, 1024, fh) == NULL ) break;
+
+ if ( strncmp(line, "Final: ", 7) == 0 ) {
+ int i;
+ if ( sscanf(line, "Final: %i images processed", &i) == 1 ) {
+ n_proc = i;
+ found = 1;
+ }
+ } else if ( strstr(line, " images processed, ") != NULL ) {
+ int i;
+ if ( sscanf(line, "%i ", &i) == 1 ) {
+ n_proc = i;
+ found = 1;
+ }
+ }
+
+ } while ( 1 );
+
+ } while ( !found && (len < 16384) );
+
+ if ( !found ) {
+ ERROR("Couldn't find status message in last 16k of %s - "
+ "indexamajig output probably contains copious errors.\n",
+ filename);
+ }
fclose(fh);