NyxErinys/crunchy-cli
1
0
Fork 0
mirror of https://github.com/crunchy-labs/crunchy-cli.git synced 2026-01-21 04:02:00 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

rename merge-auto-tolerance to merge-time-tolerance (#391)

Browse source
This commit is contained in:
Simon 2024-04-26 15:50:44 +02:00 • committed by GitHub
parent cf8bfb02ac
commit bf28dbf1ce
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

6
README.md
View file

@ -471,13 +471,13 @@ The `archive` command lets you download episodes with multiple audios and subtit
Default is `auto`. Default is `auto`.
- <span id="archive-merge-auto-tolerance">Merge auto tolerance</span> - <span id="archive-merge-time-tolerance">Merge time tolerance</span>
Sometimes two video tracks are downloaded with `--merge` set to `auto` even if they only differ some milliseconds in length which shouldn't be noticeable to the viewer. Sometimes two video tracks are downloaded with `--merge` set to `auto` even if they only differ some milliseconds in length which shouldn't be noticeable to the viewer.
To prevent this, you can specify a range in milliseconds with the `--merge-auto-tolerance` flag that only downloads one video if the length difference is in the given range. To prevent this, you can specify a range in milliseconds with the `--merge-time-tolerance` flag that only downloads one video if the length difference is in the given range.
```shell ```shell
$ crunchy-cli archive -m auto --merge-auto-tolerance 100 https://www.crunchyroll.com/series/GY8VEQ95Y/darling-in-the-franxx $ crunchy-cli archive -m auto --merge-time-tolerance 100 https://www.crunchyroll.com/series/GY8VEQ95Y/darling-in-the-franxx
``` ```
Default are `200` milliseconds. Default are `200` milliseconds.

4
crunchy-cli-core/src/archive/command.rs
View file

@ -104,7 +104,7 @@ pub struct Archive {
help = "If the merge behavior is 'auto', only download multiple video tracks if their length difference is higher than the given milliseconds" help = "If the merge behavior is 'auto', only download multiple video tracks if their length difference is higher than the given milliseconds"
)] )]
#[arg(long, default_value_t = 200)] #[arg(long, default_value_t = 200)]
pub(crate) merge_auto_tolerance: u32, pub(crate) merge_time_tolerance: u32,
#[arg(help = "Tries to sync the timing of all downloaded audios to match one video")] #[arg(help = "Tries to sync the timing of all downloaded audios to match one video")]
#[arg( #[arg(
long_help = "Tries to sync the timing of all downloaded audios to match one video. \ long_help = "Tries to sync the timing of all downloaded audios to match one video. \
@ -577,7 +577,7 @@ async fn get_format(
.sub(single_format.duration) .sub(single_format.duration)
.abs() .abs()
.num_milliseconds() .num_milliseconds()
< archive.merge_auto_tolerance.into() => < archive.merge_time_tolerance.into() =>
{ {
// If less than `audio_error` apart, use same audio. // If less than `audio_error` apart, use same audio.
closest_format closest_format