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

ERROR: stopping task which is already stopped #59

Closed
aereaux opened this issue Oct 29, 2016 · 21 comments
Closed

ERROR: stopping task which is already stopped #59

aereaux opened this issue Oct 29, 2016 · 21 comments
Labels
Bug Generic bug: can be used together with more specific labels

Comments

@aereaux
Copy link

aereaux commented Oct 29, 2016

When ripping a certain track, I get this error:

ERROR: stopping task which is already stopped
  File "/usr/bin/rip", line 11, in <module>
    load_entry_point('whipper==0.3.0', 'console_scripts', 'rip')()
  File "/usr/lib/python2.7/site-packages/morituri/rip/main.py", line 22, in main
    ret = c.parse(sys.argv[1:])
  File "/usr/lib/python2.7/site-packages/morituri/rip/main.py", line 89, in parse
    logcommand.LogCommand.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/morituri/common/logcommand.py", line 62, in parse
    command.Command.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/morituri/extern/command/command.py", line 401, in parse
    return self.subCommands[command].parse(args[1:])
  File "/usr/lib/python2.7/site-packages/morituri/common/logcommand.py", line 62, in parse
    command.Command.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/morituri/extern/command/command.py", line 401, in parse
    return self.subCommands[command].parse(args[1:])
  File "/usr/lib/python2.7/site-packages/morituri/common/logcommand.py", line 62, in parse
    command.Command.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/morituri/extern/command/command.py", line 363, in parse
    ret = self.do(args)
  File "/usr/lib/python2.7/site-packages/morituri/rip/cd.py", line 174, in do
    self.doCommand()
  File "/usr/lib/python2.7/site-packages/morituri/rip/cd.py", line 482, in doCommand
    ripIfNotRipped(i + 1)
  File "/usr/lib/python2.7/site-packages/morituri/rip/cd.py", line 414, in ripIfNotRipped
    number, len(self.itable.tracks), extra))
  File "/usr/lib/python2.7/site-packages/morituri/common/program.py", line 603, in ripTrack
    runner.run(t)
  File "/usr/lib/python2.7/site-packages/morituri/extern/task/task.py", line 479, in run
    self._loop.run()
  File "/usr/lib/python2.7/site-packages/morituri/extern/task/task.py", line 511, in c
    callable(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/morituri/extern/task/gstreamer.py", line 151, in stop
    task.Task.stop(self)
  File "/usr/lib/python2.7/site-packages/morituri/extern/task/task.py", line 154, in stop
    import traceback; traceback.print_stack()

It happens multiple times, when the ripper retries.

@RecursiveForest
Copy link
Contributor

RecursiveForest commented Nov 1, 2016

Can you:

RIP_DEBUG=5 rip cd rip > whipper.log 2>&1
gzip whipper.log

And upload the log?

@aereaux
Copy link
Author

aereaux commented Nov 1, 2016

whipper.log.gz

@JoeLametta JoeLametta added the Bug Generic bug: can be used together with more specific labels label Nov 5, 2016
@stochashtic
Copy link

stochashtic commented Nov 13, 2016

Just installed whipper and having the same error on all cds that I try. I used to use the old morituri package.

Ripping track 1 of 16: 01. Steven Wright - And Now Little Green Bag….flac
Ripping track 1 of 16 (try 2): 01. Steven Wright - And Now Little Green Bag….flac
ERROR: stopping task which is already stopped
  File "/usr/bin/whipper", line 9, in <module>
    load_entry_point('whipper==0.4.0', 'console_scripts', 'whipper')()
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/rip/main.py", line 27, in main
    ret = c.parse(sys.argv[1:])
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/rip/main.py", line 94, in parse
    logcommand.LogCommand.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/common/logcommand.py", line 62, in parse
    command.Command.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/command/command.py", line 401, in parse
    return self.subCommands[command].parse(args[1:])
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/common/logcommand.py", line 62, in parse
    command.Command.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/command/command.py", line 401, in parse
    return self.subCommands[command].parse(args[1:])
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/common/logcommand.py", line 62, in parse
    command.Command.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/command/command.py", line 363, in parse
    ret = self.do(args)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/rip/cd.py", line 174, in do
    self.doCommand()
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/rip/cd.py", line 485, in doCommand
    ripIfNotRipped(i + 1)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/rip/cd.py", line 414, in ripIfNotRipped
    number, len(self.itable.tracks), extra))
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/common/program.py", line 603, in ripTrack
    runner.run(t)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 479, in run
    self._loop.run()
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 511, in c
    callable(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/gstreamer.py", line 151, in stop
    task.Task.stop(self)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 157, in stop
    self._notifyListeners('stopped')
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 237, in _notifyListeners
    method(self, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 373, in stopped
    self.stop()
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/program/cdparanoia.py", line 548, in stop
    task.MultiSeparateTask.stop(self)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 154, in stop
    import traceback; traceback.print_stack()
Ripping track 1 of 16 (try 3): 01. Steven Wright - And Now Little Green Bag….flac
Ripping track 1 of 16 (try 4): 01. Steven Wright - And Now Little Green Bag….flac
ERROR: stopping task which is already stopped
  File "/usr/bin/whipper", line 9, in <module>
    load_entry_point('whipper==0.4.0', 'console_scripts', 'whipper')()
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/rip/main.py", line 27, in main
    ret = c.parse(sys.argv[1:])
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/rip/main.py", line 94, in parse
    logcommand.LogCommand.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/common/logcommand.py", line 62, in parse
    command.Command.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/command/command.py", line 401, in parse
    return self.subCommands[command].parse(args[1:])
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/common/logcommand.py", line 62, in parse
    command.Command.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/command/command.py", line 401, in parse
    return self.subCommands[command].parse(args[1:])
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/common/logcommand.py", line 62, in parse
    command.Command.parse(self, argv)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/command/command.py", line 363, in parse
    ret = self.do(args)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/rip/cd.py", line 174, in do
    self.doCommand()
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/rip/cd.py", line 485, in doCommand
    ripIfNotRipped(i + 1)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/rip/cd.py", line 414, in ripIfNotRipped
    number, len(self.itable.tracks), extra))
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/common/program.py", line 603, in ripTrack
    runner.run(t)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 479, in run
    self._loop.run()
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 511, in c
    callable(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/gstreamer.py", line 151, in stop
    task.Task.stop(self)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 157, in stop
    self._notifyListeners('stopped')
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 237, in _notifyListeners
    method(self, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 373, in stopped
    self.stop()
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/program/cdparanoia.py", line 548, in stop
    task.MultiSeparateTask.stop(self)
  File "/usr/lib/python2.7/site-packages/whipper-0.4.0-py2.7.egg/morituri/extern/task/task.py", line 154, in stop
    import traceback; traceback.print_stack()
Ripping track 1 of 16 (try 5): 01. Steven Wright - And Now Little Green Bag….flac
ERROR [ 4862] "rip"                            rip               Nov 13 11:53:11      Giving up on track 1 after 5 times (morituri/rip/cd.py:423)
Giving up on track 1 after 5 times

Debug logfile:
whipper.log.gz

Look forward to the fix. Thanks.

@aereaux
Copy link
Author

aereaux commented Nov 13, 2016

As far as I can tell, this only happens with one cd (on one particular track) for me. It's not identified with MusicBrainz, but it's this on freedb, and it's the second disc of this cd. It happens wuth the 9th track.

@stochashtic
Copy link

I haven't compared our debug logs, so maybe mine is a different cause. But I am unable to rip any CD. I can move this to a new thread if advised as such by the developers.

@JoeLametta
Copy link
Collaborator

Thanks for the bug reports.

Unfortunately I still haven't got the time to analyze the logfiles (which are really verbose^). I'll post again in case of any updates.

Something which is not so clear is if this issue happens only using whipper or it affects morituri too.

^ The logger function is going to be completely rewritten to provide more meaningful data.

@aereaux
Copy link
Author

aereaux commented Nov 17, 2016

It also happens for me using morituri.

@JoeLametta
Copy link
Collaborator

It also happens for me using morituri.

OK, same error message (ERROR: stopping task which is already stopped), right?

@aereaux
Copy link
Author

aereaux commented Nov 17, 2016

Yeah. In case it matters, it happened while doing rip offset find, and it left a whole bunch of cd-paranoia processes around (I assume because it tried a bunch of other offsets after finding the first one failed).

@JoeLametta
Copy link
Collaborator

@aereaux @stochashtic

Could you repost the logfiles using an updated version of whipper (following these instructions)?

Thanks

@aereaux
Copy link
Author

aereaux commented Jan 16, 2017

Here are the updated logs. I am using whipper 0.4.2-1 from the arch linux AUR.
whipper.log.gz

@JoeLametta
Copy link
Collaborator

Here are the updated logs. I am using whipper 0.4.2-1 from the arch linux AUR.

From a quick review it seems to me that this bug may be related to GStreamer usage in whipper (which is scheduled for removal, will be done soon™). So my suggestion is to wait until that removal has been completed, then check if the issue still affects whipper.

Thanks

PS: #101 too seems to be related to the same issue.

@JoeLametta
Copy link
Collaborator

@aereaux @stochashtic

GStreamer has been dropped.
Could you try testing if this issue still occurs using whipper's latest version?

Thanks in advance,

Joe

@aereaux
Copy link
Author

aereaux commented Apr 24, 2017

The error I get now is this:

Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/morituri/extern/task/task.py", line 511, in c
    callable(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/morituri/common/checksum.py", line 49, in _crc32
    w = wave.open(self.path)
  File "/usr/lib/python2.7/wave.py", line 511, in open
    return Wave_read(f)
  File "/usr/lib/python2.7/wave.py", line 164, in __init__
    self.initfp(f)
  File "/usr/lib/python2.7/wave.py", line 131, in initfp
    raise Error, 'file does not start with RIFF id'
wave.Error: file does not start with RIFF id
exception Error at /usr/lib/python2.7/wave.py:131: initfp(): file does not start with RIFF id
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/morituri/extern/task/task.py", line 511, in c
    callable(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/morituri/common/checksum.py", line 49, in _crc32
    w = wave.open(self.path)
  File "/usr/lib/python2.7/wave.py", line 511, in open
    return Wave_read(f)
  File "/usr/lib/python2.7/wave.py", line 164, in __init__
    self.initfp(f)
  File "/usr/lib/python2.7/wave.py", line 131, in initfp
    raise Error, 'file does not start with RIFF id'
Error: file does not start with RIFF id

@JoeLametta
Copy link
Collaborator

The error I get now is this:

Thanks, could you also provide a debug logfile (instructions HERE)?

@aereaux
Copy link
Author

aereaux commented Apr 24, 2017

Here. This was done using commit 17b70ce.
whipper.log.gz

@stochashtic
Copy link

Works fine for me now.

@aereaux
Copy link
Author

aereaux commented May 21, 2017

@stochashtic What version are you using?

@stochashtic
Copy link

stochashtic commented May 21, 2017

Sorry, I am using commit 5b2639a

@aereaux
Copy link
Author

aereaux commented May 21, 2017

Unfortunately I still get the RIFF id error with the most recent git commit (gb331f53)

@aereaux
Copy link
Author

aereaux commented Aug 22, 2017

I think that the RIFF id is a separate problem (#136). As far as I can tell, the original problem is fixed, so I'm closing this.

@aereaux aereaux closed this as completed Aug 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Generic bug: can be used together with more specific labels
Projects
None yet
Development

No branches or pull requests

4 participants