Skip to content
This repository has been archived by the owner on Jul 4, 2024. It is now read-only.

Commit

Permalink
rename merge-auto-tolerance to merge-time-tolerance (#391)
Browse files Browse the repository at this point in the history
  • Loading branch information
Frooastside committed Apr 26, 2024
1 parent cf8bfb0 commit bf28dbf
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -471,13 +471,13 @@ The `archive` command lets you download episodes with multiple audios and subtit

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.
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
$ 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.
Expand Down
4 changes: 2 additions & 2 deletions crunchy-cli-core/src/archive/command.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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"
)]
#[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(
long_help = "Tries to sync the timing of all downloaded audios to match one video. \
Expand Down Expand Up @@ -577,7 +577,7 @@ async fn get_format(
.sub(single_format.duration)
.abs()
.num_milliseconds()
< archive.merge_auto_tolerance.into() =>
< archive.merge_time_tolerance.into() =>
{
// If less than `audio_error` apart, use same audio.
closest_format
Expand Down

0 comments on commit bf28dbf

Please sign in to comment.