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

Configurable diff to decide when a frame is orange #21

Closed
wants to merge 1 commit into from
Closed

Configurable diff to decide when a frame is orange #21

wants to merge 1 commit into from

Conversation

soulgalore
Copy link
Contributor

Using Chrome on Ubuntu and Mac OS X recording a video in 60fps using ffmpeg
it sometimes happens that the first frame after the full orange frame is
partly orange (half or 1/4 orange etc). This only happens for me with Chrome.
See #20. VisualMetrics then picks up that partly orange frame as the first
visual change.

The current default config only makes the difference between a full orange and
white. Making it configurable makes it possible to have a higher limit when you
run Chrome.

I got a bunch of example videos when this happens if need them Pat? It would be nicer to have a solution that fix the problem permanently but this at least makes it possible to have different configs that avoids picking up a too early visual change on Chrome.

Using Chrome on Ubuntu and Mac OS X recording a video in 60fps using ffmpeg
it sometimes happens that the first frame after the full orange frame is
partly orange (half or 1/4 orange etc). This only happens for me with Chrome.
See #20. VisualMetrics then picks up that partly orange frame as the first
visual change.

The current default config only makes the difference between a full orange and
white. Making it configurable makes it possible to have a higher limit when you
run Chrome.
@soulgalore
Copy link
Contributor Author

No, that will only solve some cases, have examples where it doesn't help. Let aim for a solution that fix every case.

@soulgalore soulgalore closed this Jan 9, 2017
@soulgalore
Copy link
Contributor Author

Removing the cropping --gravity Center -crop 50%x33%+0+0 before the orange check did the trick for me.

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

Successfully merging this pull request may close these issues.

1 participant