Skip to content
This repository has been archived by the owner on Dec 23, 2022. It is now read-only.

Auto-crop gives unacceptable results #146

Open
lalloni opened this issue Mar 30, 2020 · 6 comments
Open

Auto-crop gives unacceptable results #146

lalloni opened this issue Mar 30, 2020 · 6 comments

Comments

@lalloni
Copy link

lalloni commented Mar 30, 2020

On several videos I'm trying to convert to h265 using h265ize the autocrop detection gives very bad results (blackboard tutorial videos, autocrop takes out half+ of the content which is actually wanted) so I would want and option to remove autocroping but keeping the rest of the normalization levels enabled.
From what I read in the code/docs, that is not possible because for disabling autocroping I would need to say --normalize-level 0 which takes everything out, right?
I think a specific option for disable only autocroping would be useful.
Additionally/alternatively a threshold option to keep autocroping within sane limits might work too: say "--max-auto-croping .2" which would mean that if autocroping tries to kill more than 20% of pixels it should be automatically disabled.
Thoughts?

@spoo333
Copy link

spoo333 commented Apr 3, 2020

support your request!!

@Kazz3r24
Copy link

Kazz3r24 commented Apr 3, 2020

Using --normalize-level 0 has worked flawlessly for me to eliminate cropping. In my experience so far, setting the normalization level just leaves the video ratio as is. I've had no issues with it yet.

@lalloni
Copy link
Author

lalloni commented Apr 3, 2020

Using --normalize-level 0 has worked flawlessly for me to eliminate cropping. In my experience so far, setting the normalization level just leaves the video ratio as is. I've had no issues with it yet.

I get that. That's exactly how I solved the unwanted cropping.

What I'm asking is being able to disable auto-cropping only but keeping the rest of the stuff that comes with higher values of --normalization-level:

  • various audio normalization alternatives
  • video deinterlacing
  • audio stream title handling

That's from a quick search on https://github.com/FallingSnow/h265ize/blob/master/lib/classes/video.js

@spoo333
Copy link

spoo333 commented Apr 9, 2020 via email

@lalloni
Copy link
Author

lalloni commented Apr 9, 2020

AFAIK you need to use --normalize-level 0 to disable auto-cropping and that disables everything else that is controlled with that flag.

@spoo333
Copy link

spoo333 commented Apr 10, 2020

It appears indeed that a few features are left out with --normalize-level 0. Fine for videos without subs ...

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

No branches or pull requests

3 participants