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

support nvidia capture SDK v7 #1822

Closed
totaam opened this issue Apr 28, 2018 · 6 comments
Closed

support nvidia capture SDK v7 #1822

totaam opened this issue Apr 28, 2018 · 6 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Apr 28, 2018

Issue migrated from trac ticket # 1822

component: encodings | priority: major | resolution: fixed

2018-04-28 11:04:24: antoine created the issue


@totaam
Copy link
Collaborator Author

totaam commented Apr 28, 2018

2018-04-28 11:04:57: antoine changed status from new to assigned

@totaam
Copy link
Collaborator Author

totaam commented Apr 28, 2018

2018-04-28 11:04:57: antoine edited the issue description

@totaam
Copy link
Collaborator Author

totaam commented Apr 28, 2018

2018-04-28 14:30:41: antoine commented


  • linux API changes in r19097: add BGRA pixel buffer support
  • win32 API changes in r19098: new constants

@totaam
Copy link
Collaborator Author

totaam commented Apr 29, 2018

2018-04-29 07:17:23: antoine changed status from assigned to closed

@totaam
Copy link
Collaborator Author

totaam commented Apr 29, 2018

2018-04-29 07:17:23: antoine set resolution to fixed

@totaam
Copy link
Collaborator Author

totaam commented Apr 29, 2018

2018-04-29 07:17:23: antoine commented


Noteworthy related changes:

  • r19110. CUDA image wrappers are marked as "thread safe" and we use this flag to skip freezing the picture. This works at present, but may break in the future if the hardware encoders start using the CUDA buffer directly. (it seems they copy it first?)
  • r19112: prefer gpu encoders (score boost)
  • r19113: use BGRX as pixel format, we may now be able to remove the XRGB kernels from CUDA

Tested on both Linux and Windows, with both CUDA and "SYS" backends.

@totaam totaam closed this as completed Apr 29, 2018
@totaam totaam added the v2.2.x label Jan 22, 2021
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

No branches or pull requests

1 participant