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

Using FFmpegFrameFilter with zscale and tonemapping #1389

Open
GoPenguins opened this issue Mar 21, 2020 · 11 comments
Open

Using FFmpegFrameFilter with zscale and tonemapping #1389

GoPenguins opened this issue Mar 21, 2020 · 11 comments

Comments

@GoPenguins
Copy link

When I attempt to create the following filter:

"zscale=t=linear:npl=100,format=gbrpf32le,zscale=p=bt709,tonemap=tonemap=mobius:desat=0,zscale=t=bt709:m=bt709:r=tv,format=yuv420p,scale=1920:1080"

the filter throws an exception when attempting to start:

org.bytedeco.javacv.FrameFilter$Exception: avfilter_graph_parse_ptr() error -22

The filter in question works fine from ffmpeg command line, but I'm using this filter with frame grabber. Any suggestions?

@saudet
Copy link
Member

saudet commented Mar 22, 2020 via email

@GoPenguins
Copy link
Author

Sorry, I forgot to attach... Pretty sure this is what you are looking for

No such filter: 'zscale'

@saudet
Copy link
Member

saudet commented Mar 22, 2020

Ok, so we would need to enable it in the build. Contributions are welcome, so please do send a pull request if you get it working! Let me know if you have any questions though. Thanks

@b005t3r
Copy link

b005t3r commented Aug 31, 2021

I guess this still hasn't been changed? The lack of zscale filter makes converting HDR videos impossible.

Enabling this shouldn't be that difficult:

To enable compilation of this filter, you need to configure FFmpeg with --enable-libzimg

http://underpop.online.fr/f/ffmpeg/help/zscale.htm.gz

@callisto-jovy
Copy link

callisto-jovy commented Oct 18, 2023

As converting HDR videos would be very handy and there has not been an update yet, would it be possible to include the flag in the next release?

@saudet
Copy link
Member

saudet commented Oct 18, 2023

Contributions are welcome! Please open a pull request with the change

@callisto-jovy
Copy link

Contributions are welcome! Please open a pull request with the change

Will do!

@b005t3r
Copy link

b005t3r commented Dec 15, 2023

So what's the status of this? I guess it still hasn't been resolved?

@b005t3r
Copy link

b005t3r commented Dec 15, 2023

I created a pull request for this, should fix the issue:

bytedeco/javacpp-presets#1448

@callisto-jovy
Copy link

So what's the status of this? I guess it still hasn't been resolved?

Sorry, I haven't been able to get to the issue for some time now. I'll try to fix the remaining cases next week.

@b005t3r
Copy link

b005t3r commented Jan 17, 2024

@callisto-jovy were you able to make any progress on this?

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

No branches or pull requests

4 participants