Added NVIDIA Video Codec Preset for Windows

[devjeonghwan]

Fixed #1008
Fixed #1026

[devjeonghwan]

That sample java code based on nvidia official nvidia video codec sample c++ code.

[saudet]

Thanks! I'm wondering though, could we use the header files from https://github.com/FFmpeg/nv-codec-headers?
That way we wouldn't need to download and install the SDK separately.

[devjeonghwan]

@saudet i think header files is fine. but missing nvcuvid link file.

[devjeonghwan]

ok, let's test compile with "replace official sdk header files to that repository header files" and "use official link(dll) file". and i will let you know result.

[saudet]

Yeah, that's not going to work. We'll need to do the same as TensorRT here I'm afraid:
https://github.com/bytedeco/javacpp-presets/blob/master/.github/actions/deploy-centos/action.yml#L95

[devjeonghwan]

okey, you mean "write 'download video sdk and unzip' code to each platform github action", right?

[saudet]

Yes, it's only Linux and Windows, but it's still annoying.

[devjeonghwan]

i will upload each platform nvcodec SDK files to google driver and modify github action script.

btw, why you didn't check downloaded file hash check? isn't dangerous? (if i upload modulated dll to google driver and i link it...? it is terrible)

[saudet]

AFAIK, there is only 1 zip for all platforms, but it needs to be installed for each, yes. Sure, we can add hash checks, that would be a good thing, contributions are welcome! Please open another PR to do that though.

[devjeonghwan]

I added git actions and fix small sample problem.

[saudet]

Please add a workflow for GitHub Actions here to get this tested:
https://github.com/bytedeco/javacpp-presets/tree/master/.github/workflows

[devjeonghwan]

Sorry, i added

[devjeonghwan]

Ah... something happen.. should i have to change nvcodec dependency javacpp version 1.5.5-snap to 1.5.6-snap?

[devjeonghwan]

oops.. i forgot linux support code in pom.xml... so i commited.... xD but i don't know why windows build failure

[devjeonghwan]

i will close this pr and check again.

[devjeonghwan]

oh.. i found reason.. i think we need merge nvcodec action file to bytedeco/javacpp-presets@action branche

[devjeonghwan]

I pushed

[devjeonghwan]

Fixed #1026

[devjeonghwan]

Sorry for force push. i pushed a file I didn't care about by my mistake.

BTW, @saudet i have to push tensorrt javadoc encoding config(#1026) to this PR? or i make another PR?

[saudet]

Feel free to include the fix for TensorRT in this PR. Thanks!

[devjeonghwan]

Nice!

[saudet]

Hey, I'd like to fix a few nits before merging this. Can you give me push access to this branch?

[devjeonghwan]

Invited!

[saudet]

Ok, this looks good to merge! Let me know if this looks to you.

[devjeonghwan]

@saudet that goes without saying! nice!