Runtime variable was overwritten and then incorrectly used by ETA code
This resulted into ETA to forget hours of runtime and calculate incorrect average_speed
This commit is contained in:
Lukáš Horáček 2021-10-02 13:02:14 +02:00
parent d15b882929
commit 0a8dced82e
No known key found for this signature in database
GPG key ID: 5A1A3D4E71EF906F

View file

@ -685,13 +685,13 @@ class DLManager(Process):
total_used = (num_shared_memory_segments - total_avail) * (self.analysis.biggest_chunk / 1024 / 1024)
if runtime and processed_chunks:
rt_hours, runtime = int(runtime // 3600), runtime % 3600
rt_minutes, rt_seconds = int(runtime // 60), int(runtime % 60)
rt_hours, runtime_without_hours = int(runtime // 3600), runtime % 3600
rt_minutes, rt_seconds = int(runtime_without_hours // 60), int(runtime_without_hours % 60)
average_speed = processed_chunks / runtime
estimate = (num_chunk_tasks - processed_chunks) / average_speed
hours, estimate = int(estimate // 3600), estimate % 3600
minutes, seconds = int(estimate // 60), int(estimate % 60)
hours, estimate_without_hours = int(estimate // 3600), estimate % 3600
minutes, seconds = int(estimate_without_hours // 60), int(estimate_without_hours % 60)
else:
hours = minutes = seconds = 0
rt_hours = rt_minutes = rt_seconds = 0