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

Merge not working #42

Open
Artrix opened this issue May 24, 2022 · 1 comment
Open

Merge not working #42

Artrix opened this issue May 24, 2022 · 1 comment

Comments

@Artrix
Copy link

Artrix commented May 24, 2022

I get this error on Python 3.9:

2022-05-24 20:10:28,887 - INFO - download.spO_hiEoeAM - No missing segment detected. All good.
2022-05-24 20:10:29,067 - DEBUG - download.spO_hiEoeAM - Probed "0000000000_video.ts". codec_name: h264, duration: 0.567, start_time: 0.0.
2022-05-24 20:10:29,185 - DEBUG - download.spO_hiEoeAM - Probed "0000000000_audio.ts". codec_name: aac, duration: 0.580499, start_time: 0.0.
2022-05-24 20:10:29,186 - INFO - download.spO_hiEoeAM - Performing concat method NativeConcatFile
2022-05-24 20:10:29,186 - ERROR - download.spO_hiEoeAM - unsupported operand type(s) for /: 'str' and 'str'
Traceback (most recent call last):
  File "/home/artrix/livestream_saver/livestream_saver/merge.py", line 553, in merge
    concat_video_file = methods[attempt](
  File "/home/artrix/livestream_saver/livestream_saver/merge.py", line 296, in __init__
    super().__init__(*args, **kwargs)
  File "/home/artrix/livestream_saver/livestream_saver/merge.py", line 73, in __init__
    self._final_file: Path = output_dir / \
TypeError: unsupported operand type(s) for /: 'str' and 'str'
2022-05-24 20:10:29,186 - ERROR - livestream_saver - Missing concat video file: None
Traceback (most recent call last):
  File "/home/artrix/livestream_saver/livestream_saver.py", line 963, in main
    error = args["func"](config, args)
  File "/home/artrix/livestream_saver/livestream_saver.py", line 636, in merge_mode
    written_file = merge(
  File "/home/artrix/livestream_saver/livestream_saver/merge.py", line 611, in merge
    raise Exception(f"Missing concat video file: {concat_video_file}")
Exception: Missing concat video file: None```
glubsy added a commit that referenced this issue May 24, 2022
Workaround for #42.
If output_dir is initialized to None (for whatever reason, could be from
a CLI argument, or bogus config file override) fall back to using
current working directory to store the concat file.
@glubsy
Copy link
Owner

glubsy commented May 24, 2022

Thanks for reporting. Looks like an issue in either your config file or a CLI argument.

I suspect you might have specified "output_dir" somewhere to some blank value? I can only speculate, but if you share your config file and arguments you used, I could better point out what caused the issue.

For now I have added a workaround in c543a5a which falls back to storing the concat file in the current working directory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants