Skip to content

Commit dfcc0b6

Browse files
authored
[Fix] Fix bug in tools/analyse_logs.py caused by wrong plot_iter in some cases. (open-mmlab#1428)
* fix_analyze_logs * fix comments * fix comments
1 parent ecd2dc4 commit dfcc0b6

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

tools/analyze_logs.py

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,9 @@ def plot_curve(log_dicts, args):
3131
plot_epochs = []
3232
plot_iters = []
3333
plot_values = []
34-
# In some log files, iters number is not correct, `pre_iter` is
35-
# used to prevent generate wrong lines.
36-
pre_iter = -1
34+
# In some log files exist lines of validation,
35+
# `mode` list is used to only collect iter number
36+
# of training line.
3737
for epoch in epochs:
3838
epoch_logs = log_dict[epoch]
3939
if metric not in epoch_logs.keys():
@@ -43,11 +43,9 @@ def plot_curve(log_dicts, args):
4343
plot_values.append(epoch_logs[metric][0])
4444
else:
4545
for idx in range(len(epoch_logs[metric])):
46-
if pre_iter > epoch_logs['iter'][idx]:
47-
continue
48-
pre_iter = epoch_logs['iter'][idx]
49-
plot_iters.append(epoch_logs['iter'][idx])
50-
plot_values.append(epoch_logs[metric][idx])
46+
if epoch_logs['mode'][idx] == 'train':
47+
plot_iters.append(epoch_logs['iter'][idx])
48+
plot_values.append(epoch_logs[metric][idx])
5149
ax = plt.gca()
5250
label = legend[i * num_metrics + j]
5351
if metric in ['mIoU', 'mAcc', 'aAcc']:

0 commit comments

Comments
 (0)