Skip to content
Snippets Groups Projects
Commit 56c996bb authored by Robert Goldmann's avatar Robert Goldmann
Browse files

SpotifyRecorder: fixed log messages

parent 29a6f060
Branches
Tags
No related merge requests found
...@@ -63,6 +63,9 @@ class SpotifyAudioRecorder(threading.Thread): ...@@ -63,6 +63,9 @@ class SpotifyAudioRecorder(threading.Thread):
if not self._stopEvent.is_set(): if not self._stopEvent.is_set():
self._stopEvent.set() self._stopEvent.set()
def is_stopped(self) -> bool:
return self._stopEvent.is_set()
if __name__ == '__main__': if __name__ == '__main__':
recorder = SpotifyAudioRecorder( recorder = SpotifyAudioRecorder(
......
...@@ -88,6 +88,7 @@ class SpotifyRecorder: ...@@ -88,6 +88,7 @@ class SpotifyRecorder:
recordedTracks = [] recordedTracks = []
skippedTracks = [] skippedTracks = []
errorTracks = []
for index, track in enumerate(tracks[:2]): for index, track in enumerate(tracks[:2]):
if track['is_local']: if track['is_local']:
# TODO: # TODO:
...@@ -100,7 +101,7 @@ class SpotifyRecorder: ...@@ -100,7 +101,7 @@ class SpotifyRecorder:
LOGGER.info(f'Recording track {index + 1}/{len(tracks)}: "{track["track"]["name"]}"...') LOGGER.info(f'Recording track {index + 1}/{len(tracks)}: "{track["track"]["name"]}"...')
recorder = None recorder = None
try: try:
filePath = self.__determine_file_path(index, track) filePath = self.__determine_file_path(index + 1, track)
recorder = SpotifyAudioRecorder(self._audioDeviceName, filePath) recorder = SpotifyAudioRecorder(self._audioDeviceName, filePath)
recorder.start() recorder.start()
...@@ -113,12 +114,12 @@ class SpotifyRecorder: ...@@ -113,12 +114,12 @@ class SpotifyRecorder:
recordedTracks.append(track['track']['name']) recordedTracks.append(track['track']['name'])
except Exception as e: except Exception as e:
LOGGER.error(f'An error occurred while recording track "{track["track"]["name"]}"', exc_info=e) LOGGER.error(f'An error occurred while recording track "{track["track"]["name"]}"', exc_info=e)
skippedTracks.append(track['track']['name']) errorTracks.append(track['track']['name'])
if recorder is not None and recorder.is_alive(): if recorder is not None and not recorder.is_stopped():
recorder.stop() recorder.stop()
LOGGER.info('### DONE ###') LOGGER.info('### DONE ###')
LOGGER.info(f'{len(tracks)} tracks, {len(recordedTracks)} recorded, {len(skippedTracks)} skipped') LOGGER.info(f'{len(tracks)} tracks, {len(recordedTracks)} recorded, {len(skippedTracks)} skipped, {len(errorTracks)} errors')
def __determine_file_path(self, index: int, track) -> str: def __determine_file_path(self, index: int, track) -> str:
artists = track['track']['artists'] artists = track['track']['artists']
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment