Ryan Foster
Ryan Foster
Prior discussion occurred here: * #10600
Just FYI, CUDA was originally added here: https://github.com/obsproject/obs-studio/commit/a3fface27fc603ef1dded179ae87213f2da8ca3f It was removed here: https://github.com/obsproject/obs-studio/commit/86572935afda66442ea2a24c47d1dcca21185d97
> Can this change be incorporated in the latest obs version? We always investigate dependency updates between versions.
> Th PR title is misleading and I think some of the comments on the code. > > While the intention may be to add configure multitrack video, this PR...
> info: [obs-nvenc] nv12/p010 not active, falling back to non-texture encoder Please see #10567.
I converted the GIF to an MP4. https://github.com/obsproject/obs-studio/assets/624931/babb979e-3572-415a-b74f-bbd792a02beb I cannot reproduce this with a secondary display when both displays are at 100% scaling.
> I would not consider changing the screen scale a fix as 175% I wasn't suggesting this. I was merely documenting what does _not_ reproduce the issue to illustrate that...
~~I have no more nits at this time.~~ The commit message body still says `obs` in lowercase at the end of the second sentence. Please capitalize it. Will need someone...
With https://github.com/obsproject/obs-studio/pull/10494 merged, this can be rebased and undrafted.
Off-thread discussion indicates that this looks good. Squash the commits appropriately and we'll give it a (hopefully) final pass.