RupertLuo / Valley

The official repository of "Video assistant towards large language model makes everything easy"
199 stars 13 forks source link

The command in`cmd_list` is confusing. #29

Open twotwoiscute opened 9 months ago

twotwoiscute commented 9 months ago
yt-dlp --merge-output-format mp4 "https://www.youtube.com/watch?v=GCfvy2hUB6U" -o "VATEX/v_GCfvy2hUB6U.mp4" || mv "VATEX/v_GCfvy2hUB6U.mp4.part" "VATEX/v_GCfvy2hUB6U.mp4"

yt-dlp --merge-output-format mp4 "https://www.youtube.com/watch?v=kSi8SwNtg8k" -o "VATEX/v_kSi8SwNtg8k.mp4" || mv "VATEX/v_kSi8SwNtg8k.mp4.part" "VATEX/v_kSi8SwNtg8k.mp4"

yt-dlp --merge-output-format mp4 "https://www.youtube.com/watch?v=ZNwakAqKZVc" -o "VATEX/v_ZNwakAqKZVc.mp4" || mv "VATEX/v_ZNwakAqKZVc.mp4.part" "VATEX/v_ZNwakAqKZVc.mp4"

yt-dlp --merge-output-format mp4 "https://www.youtube.com/watch?v=6x3sD--s79E" -o "VATEX/v_6x3sD--s79E.mp4" || mv "VATEX/v_6x3sD--s79E.mp4.part" "VATEX/v_6x3sD--s79E.mp4"

yt-dlp --merge-output-format mp4 "https://www.youtube.com/watch?v=WtS1KktMHD8" -o "VATEX/v_WtS1KktMHD8.mp4" || mv "VATEX/v_WtS1KktMHD8.mp4.part" "VATEX/v_WtS1KktMHD8.mp4"

This is part of content I get from cmd_listfor downloading video from VATEX, the command indicated that if download fails, rename it from .part file to .mp4 file May I ask the reason for doing this? At least from my perspective, this is mixing the complete videos and incomplete videos together, in the other word, there's no way to tell which one fails for postprocesssing if needed.

RupertLuo commented 9 months ago

When I downloaded it, I found that I could only download the .part file, but if I changed the suffix to mp4, the file could be played normally.