-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
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
obs-filters: Add panic hotkeys to delay filters #1372
Conversation
Adds obs_hotkey_register_filter
Add a panic hotkey for dumping frames
Add panic hotkey
014333c
to
4d2e823
Compare
For those who feel the description is insufficient, this feature is basically related to https://ideas.obsproject.com/ideas/33/add-a-shortcut-to-clear-the-buffer-before-it-leaves-obs-to-prevent-streaming-bad-things I do worry about UX and discoverability for this kind of feature, but have no idea what the best way to expose it would be. |
The GPU delay filter is meant for minor sync adjustments, it's not really suitable for a dump buffer. It buffers raw video data, so if someone uses a capture card this new default of 20 seconds requires over 4 GB of VRAM and would likely cause major issues for users with lower end GPUs. I think the correct place to implement this would be the delay buffer, where already encoded video packets can be dumped instead. |
@notr1ch the video frames are stored locally on the cpu (unless I'm mistaken, I'm probably mistaken stagesurf wasn't used), but you're right, it's quite hefty in terms of memory consumption. As for the right place to implement this, I went w/ the already existing filters, although I would definitely consider the delay buffer as well. |
I do not think this is the best way to approach this problem right now. I like the idea of having panic keys, but right now, there are issues with hotkeys on filters (see my comment on #1508 for more details). I really do not want to do this right now. This is a solvable problem, but I think the issues with hotkeys on filters needs to be taken care of before we start proposing hotkeys for filters. Besides, I don't really want to modify the GPU/async delay filters to do this. I think it might be more ideal as a separate filter or something. But right now, let's put this aside for more important priorities. Wait until after the clang-format processing before continuing any new proposals for this. |
Hi! Pardon the necro-bumping here, but it's been a few years. Is there any update on this PR? I'd love to see this feature in OBS |
To show your support for this feature, vote for it on our ideas page: https://ideas.obsproject.com/posts/33/add-dump-button-functionality-to-obs That said, this PR won't be re-opened. It would have to be implemented a different way. |
Adds a panic hotkey to async and gpu delay filters.