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

Doesn't work VaapiPostProcessScaler #817

Open
AlexanderKomarov opened this issue Dec 8, 2017 · 9 comments
Open

Doesn't work VaapiPostProcessScaler #817

AlexanderKomarov opened this issue Dec 8, 2017 · 9 comments

Comments

@AlexanderKomarov
Copy link

AlexanderKomarov commented Dec 8, 2017

VaapiPostProcessScaler doesn't work. It returns SharedPtr < VideoFrame > with zeros in crop.width and crop.height. I tested it on the vpp test and my application. I use DRM as display.
But all tests are passed.

I use Ubuntu 16.04. The latest libva, libyami and intel-vaapi-driver.

@xuguangxin
Copy link
Contributor

could you share you command line and try our yamivpp application https://github.com/01org/libyami-utils/blob/master/tests/vpp.cpp like "yamivpp input_1920x1080.i420 output_320x240.yv12"

@AlexanderKomarov
Copy link
Author

I just run vpp/unittest. Also I added this line std::cout << std::to_string(dest->crop.width) << " " << std::to_string(dest->crop.height) << std::endl; to VppDeinterlace function in vaapipostprocess_scaler_unittest.cpp after this line EXPECT_EQ(YAMI_SUCCESS, scaler.process(src, dest));
And I get only zeros. Is it normal?

I ran our yamivpp application. And it works.

@xuguangxin
Copy link
Contributor

xuguangxin commented Dec 13, 2017

Yes, it's defined feature. If crop is zero, It will scale to the entire surface.
We have src crop and dest crop. src crop means you only need a sub rect on the source, dest crop means you only need scale src to the sub rect of dest.

@AlexanderKomarov
Copy link
Author

ok, thank you for the answer.

Also quick question I tried to test YAMI_FOURCC_Y800 format for scaling but I get the error: "the requested function is not implemented". Do you plan to support it? And where can I get the list of supported formats for scaling?

@xuguangxin
Copy link
Contributor

It's the error returned by libva. we depend on libva to do vpp.

Hi @xhaihao,
Can we query csc format in libva?

HI @AlexanderKomarov
Could you share us the usecase? Why you need Y800 scaling?

thanks

@xhaihao
Copy link

xhaihao commented Dec 14, 2017

vaQuerySurfaceAttributes can return the supported surface attribute list for a given config, including pixel formats

@AlexanderKomarov
Copy link
Author

@xuguangxin @xhaihao thanks for the answers.

We try to use Intel GPU for decoding/encoding JPEG from camera, scaling and transforming images. We use gray image in our system. Our system does face detection and face recognition.

@AlexanderKomarov
Copy link
Author

@xuguangxin @xhaihao maybe is there some sense to add the issue of adding support Y800 for vpp in libva project?

@xuguangxin
Copy link
Contributor

@xhaihao , what's the suggestion?:)

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