Skip to content
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.

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

More precise remaining time calculation, ui.py split #42

Draft
wants to merge 11 commits into
base: py3k
Choose a base branch
from
Prev Previous commit
Next Next commit
conversion is marked done when skipping existing files
  • Loading branch information
sezanzeb committed Sep 6, 2020
commit 9a58bb23bef3422025a3be09210a3f36e52556a0
1 change: 1 addition & 0 deletions soundconverter/gstreamer/converter.py
Original file line number Diff line number Diff line change
Expand Up @@ -426,6 +426,7 @@ def run(self):
logger.info('output file already exists, skipping \'{}\''.format(
beautify_uri(self.newname)
))
self.done = True
self._conversion_done()
return

Expand Down
4 changes: 2 additions & 2 deletions soundconverter/util/taskqueue.py
Original file line number Diff line number Diff line change
Expand Up @@ -122,8 +122,8 @@ def task_done(self, task):
"""
if task.get_progress()[0] != 1:
raise ValueError(
f'Task with a progress of {task.get_progress()[0]} called '
'task_done'
f'{task.__class__.__name__} Task with a progress of '
f'{task.get_progress()[0]} called task_done'
)
self.done.append(task)
if task not in self.running:
Expand Down