imgui_test_engine icon indicating copy to clipboard operation
imgui_test_engine copied to clipboard

Define IMGUI_VIDEO_CAPTURE_ENCODER_PATH

Open jacobfriedman opened this issue 3 years ago • 2 comments

It would be nice to have an IMGUI_VIDEO_CAPTURE_ENCODER_PATH definition included in imgui_capture_tool.h much like IMGUI_CAPTURE_DEFAULT_VIDEO_PARAMS[...]. In my world it would be conveniently sourced during ImGuiTestEngine_Postswap;

struct IMGUI_API ImGuiCaptureContext would contain char* VideoCaptureEncoderPath = IMGUI_VIDEO_CAPTURE_ENCODER_PATH or along those lines so we don't have to input it via the UI.

Thoughts?

jacobfriedman avatar Jan 16 '23 17:01 jacobfriedman

As a sidenote, IMGUI_CAPTURE_DEFAULT_VIDEO_PARAMS_FOR_FFMPEG should probably have the suffix dropped.

jacobfriedman avatar Jan 16 '23 17:01 jacobfriedman

Hello,

struct IMGUI_API ImGuiCaptureContext would contain char* VideoCaptureEncoderPath = IMGUI_VIDEO_CAPTURE_ENCODER_PATH or along those lines so we don't have to input it via the UI.

If you want to change the default value or force the value, you can write to test_io.VideoCaptureEncoderPath[] ? If you ship or use a software that has an ffmpeg executable you can should setup that path yourself in the code. We don't ship ffmpeg.exe in neither Test Suite or Test Engine so it wouldn't make sense to set a default.

As a sidenote, IMGUI_CAPTURE_DEFAULT_VIDEO_PARAMS_FOR_FFMPEG should probably have the suffix dropped.

I don't understand either. They are specifically default params in the case of using FFMPEG as an encoder. We don't assume the encoder is FFMPEG but provide default params for that case.

ocornut avatar Feb 23 '23 15:02 ocornut