Fix video context...

Author: ManifoldFR

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 
 - bindings/python : expose video recorder settings
+- bindings/python : add function `hasFfmpegSupport()` to check if... library was built with FFmpeg support
 - pixi : separate `pinocchio` and `ffmpeg` features, add to CI build matrix
 
 ### Changed

bindings/python/candlewick/video_context.py

@@ -1,15 +1,7 @@
 from contextlib import contextmanager
-from . import Visualizer
+from . import Visualizer, hasFfmpegSupport
 import warnings
 
-try:
-    import importlib
-
-    importlib.import_module("VideoRecorderSettings", package=".")
-    CANDLEWICK_HAS_VIDEO_RECORDER = True
-except ImportError:
-    CANDLEWICK_HAS_VIDEO_RECORDER = False
-
 _DEFAULT_VIDEO_SETTINGS = {"fps": 30, "bitRate": 3_000_000}
 
 __all__ = ["create_recorder_context"]
@@ -23,7 +15,7 @@ def create_recorder_context(
     fps: int = _DEFAULT_VIDEO_SETTINGS["fps"],
     bitRate: int = _DEFAULT_VIDEO_SETTINGS["bitRate"],
 ):
-    if not CANDLEWICK_HAS_VIDEO_RECORDER:
+    if not hasFfmpegSupport():
         warnings.warn(
             "This context will do nothing, as Candlewick was built without video recording support."
         )

bindings/python/src/module.cpp

@@ -26,6 +26,14 @@ BOOST_PYTHON_MODULE(pycandlewick) {
   }
   bp::def("setShadersDirectory", &setShadersDirectory, ("path"_a));
   bp::def("currentShaderDirectory", &currentShaderDirectory);
+  bp::def(
+      "hasFfmpegSupport", +[] {
+#ifdef CANDLEWICK_WITH_FFMPEG_SUPPORT
+        return true;
+#else
+    return false;
+#endif
+      });
 
   // Register SDL_Quit() as a function to call when interpreter exits.
   Py_AtExit(SDL_Quit);