Skip to content

Added property long_eta #59

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Added property long_eta #59

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
d9a9091
Added property long eta
awiebe Nov 7, 2018
627bd95
Add return to longg eta
awiebe Nov 7, 2018
5d00ada
td form of long_eta
awiebe Nov 7, 2018
dba52cb
Fix divide by zero long average
awiebe Nov 7, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions progress/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,10 @@ def elapsed(self):
@property
def elapsed_td(self):
return timedelta(seconds=self.elapsed)

@property
def long_avg(self):#Gets the total average instead of just the running average
return self.elapsed/(self.index+1)

def update_avg(self, n, dt):
if n > 0:
Expand Down Expand Up @@ -135,6 +139,13 @@ def eta(self):
@property
def eta_td(self):
return timedelta(seconds=self.eta)

@property
def long_eta(self): #For long running tasks the total average is more stable
return int(ceil(self.long_avg * self.remaining))
@property
def eta_td(self):
return timedelta(seconds=self.long_eta)

@property
def percent(self):
Expand Down