Remove WebGL renderer (#2630)

* Remove WebGL renderer

Fixes #2629

* Remove occurence of webgl_renderer in CI config

* Fix Dockerfile

* Remove reference to webgl renderer in config docs

* Update poetry lock file

* Remove some scripts related to webgl

* Don't use rich 12.0 version

* Ignore rich 12.0.0

* Fix dependency, it was wrong previously

Author: naveen521kk

.github/workflows/ci.yml

@@ -45,7 +45,7 @@ jobs:
 
       - name: Install Manim
         run: |
-          poetry install -E webgl_renderer
+          poetry install
 
       - name: Run tests
         run: |
@@ -202,7 +202,7 @@ jobs:
       - name: Install manim
         run: |
           poetry config experimental.new-installer false
-          poetry install -E webgl_renderer
+          poetry install
 
       - name: Run tests
         run: |

docker/Dockerfile

@@ -30,7 +30,7 @@ RUN wget -O /tmp/install-tl-unx.tar.gz http://mirror.ctan.org/systems/texlive/tl
 # clone and build manim
 COPY . /opt/manim
 WORKDIR /opt/manim
-RUN pip install --no-cache .[jupyterlab,webgl_renderer]
+RUN pip install --no-cache .[jupyterlab]
 
 RUN pip install -r docs/requirements.txt
 

docs/source/tutorials/configuration.rst

@@ -360,7 +360,6 @@ A list of all config options
    'progress_bar', 'quality', 'right_side', 'save_as_gif', 'save_last_frame',
    'save_pngs', 'scene_names', 'show_in_file_browser', 'sound', 'tex_dir',
    'tex_template', 'tex_template_file', 'text_dir', 'top', 'transparent',
-   'upto_animation_number', 'use_opengl_renderer', 'use_webgl_renderer',
-   'verbosity', 'video_dir', 'webgl_renderer_path', 'window_position',
-   'window_monitor', 'window_size', 'write_all', 'write_to_movie', 'enable_wireframe',
-   'force_window']
+   'upto_animation_number', 'use_opengl_renderer', 'verbosity', 'video_dir',
+   'window_position', 'window_monitor', 'window_size', 'write_all', 'write_to_movie',
+   'enable_wireframe', 'force_window']

manim/__init__.py

@@ -27,8 +27,6 @@
         config.renderer = parsed_renderer
     elif arg == "--use_opengl_renderer":
         config.renderer = "opengl"
-    elif arg == "--use_webgl_renderer":
-        config.renderer = "webgl"
 
 # many scripts depend on this -> has to be loaded first
 from .utils.commands import *  # isort:skip

manim/_config/default.cfg

@@ -93,15 +93,9 @@ tex_dir = {media_dir}/Tex
 text_dir = {media_dir}/texts
 partial_movie_dir = {video_dir}/partial_movie_files/{scene_name}
 
-# --renderer [cairo|opengl|webgl]
+# --renderer [cairo|opengl]
 renderer = cairo
 
-# --use_webgl_renderer
-use_webgl_renderer = False
-
-# --webgl_renderer_path
-webgl_renderer_path =
-
 # --use_opengl_renderer
 use_opengl_renderer = False
 

manim/_config/utils.py

@@ -255,7 +255,6 @@ class MyScene(Scene):
         "input_file",
         "media_embed",
         "media_width",
-        "webgl_renderer_path",
         "log_dir",
         "log_to_file",
         "max_files_cached",
@@ -282,7 +281,6 @@ class MyScene(Scene):
         "upto_animation_number",
         "renderer",
         "use_opengl_renderer",
-        "use_webgl_renderer",
         "enable_gui",
         "gui_location",
         "use_projection_fill_shaders",
@@ -537,7 +535,6 @@ def digest_parser(self, parser: configparser.ConfigParser) -> ManimConfig:
             "flush_cache",
             "custom_folders",
             "use_opengl_renderer",
-            "use_webgl_renderer",
             "enable_gui",
             "fullscreen",
             "use_projection_fill_shaders",
@@ -577,7 +574,6 @@ def digest_parser(self, parser: configparser.ConfigParser) -> ManimConfig:
             "movie_file_extension",
             "background_color",
             "renderer",
-            "webgl_renderer_path",
             "window_position",
         ]:
             setattr(self, key, parser["CLI"].get(key, fallback="", raw=True))
@@ -710,7 +706,6 @@ def digest_args(self, args: argparse.Namespace) -> ManimConfig:
             "renderer",
             "background_color",
             "use_opengl_renderer",
-            "use_webgl_renderer",
             "enable_gui",
             "fullscreen",
             "use_projection_fill_shaders",
@@ -1181,7 +1176,7 @@ def dry_run(self, val: bool) -> None:
 
     @property
     def renderer(self):
-        """Renderer: "cairo", "opengl", "webgl"""
+        """Renderer: "cairo", "opengl"""
         return self._d["renderer"]
 
     @renderer.setter
@@ -1221,7 +1216,7 @@ def renderer(self, val: str) -> None:
         self._set_from_list(
             "renderer",
             val,
-            ["cairo", "opengl", "webgl"],
+            ["cairo", "opengl"],
         )
 
     @property
@@ -1236,31 +1231,9 @@ def use_opengl_renderer(self, val: bool) -> None:
             self._set_from_list(
                 "renderer",
                 "opengl",
-                ["cairo", "opengl", "webgl"],
+                ["cairo", "opengl"],
             )
 
-    @property
-    def use_webgl_renderer(self):
-        """Whether or not to use WebGL renderer."""
-        return self._d["use_webgl_renderer"]
-
-    @use_webgl_renderer.setter
-    def use_webgl_renderer(self, val: bool) -> None:
-        self._d["use_webgl_renderer"] = val
-        if val:
-            self._set_from_list(
-                "webgl",
-                "renderer",
-                ["cairo", "opengl", "webgl"],
-            )
-            self["disable_caching"] = True
-
-    webgl_renderer_path = property(
-        lambda self: self._d["webgl_renderer_path"],
-        lambda self, val: self._d.__setitem__("webgl_renderer_path", val),
-        doc="Path to WebGL renderer.",
-    )
-
     media_dir = property(
         lambda self: self._d["media_dir"],
         lambda self, val: self._set_dir("media_dir", val),

manim/camera/webgl_camera.py

@@ -50,16 +50,6 @@ def render(
         )
         args["renderer"] = "opengl"
 
-    if args["use_webgl_renderer"]:
-        logger.warning(
-            "--use_webgl_renderer is deprecated, please use --renderer=webgl instead!",
-        )
-        args["renderer"] = "webgl"
-
-    if args["use_webgl_renderer"] and args["use_opengl_renderer"]:
-        logger.warning("You may select only one renderer!")
-        sys.exit()
-
     if args["save_as_gif"]:
         logger.warning("--save_as_gif is deprecated, please use --format=gif instead!")
         args["format"] = "gif"
@@ -120,20 +110,6 @@ def __repr__(self):
         except Exception:
             error_console.print_exception()
             sys.exit(1)
-    elif config.renderer == "webgl":
-        try:
-            from manim.grpc.impl import frame_server_impl
-
-            server = frame_server_impl.get(file)
-            server.start()
-            server.wait_for_termination()
-        except ModuleNotFoundError:
-            console.print(
-                "Dependencies for the WebGL render are missing. Run "
-                "pip install manim[webgl_renderer] to install them.",
-            )
-            error_console.print_exception()
-            sys.exit(1)
     else:
         for SceneClass in scene_classes_from_file(file):
             try:

manim/cli/render/commands.py

@@ -95,7 +95,7 @@ def validate_resolution(ctx, param, value):
     ),
     option(
         "--renderer",
-        type=click.Choice(["cairo", "opengl", "webgl"], case_sensitive=False),
+        type=click.Choice(["cairo", "opengl"], case_sensitive=False),
         help="Select a renderer for your Scene.",
         default=None,
     ),
@@ -105,18 +105,6 @@ def validate_resolution(ctx, param, value):
         help="Render scenes using OpenGL (Deprecated).",
         default=None,
     ),
-    option(
-        "--use_webgl_renderer",
-        is_flag=True,
-        help="Render scenes using the WebGL frontend (Deprecated).",
-        default=None,
-    ),
-    option(
-        "--webgl_renderer_path",
-        default=None,
-        type=click.Path(),
-        help="The path to the WebGL frontend.",
-    ),
     option(
         "-g",
         "--save_pngs",

manim/cli/render/render_options.py

@@ -67,7 +67,6 @@
     "GIF_FILE_EXTENSION",
     "FFMPEG_VERBOSITY_MAP",
     "VERBOSITY_CHOICES",
-    "WEBGL_RENDERER_INFO",
     "QUALITIES",
     "DEFAULT_QUALITY",
     "DEFAULT_QUALITY_SHORT",
@@ -216,12 +215,6 @@
     "CRITICAL": "fatal",
 }
 VERBOSITY_CHOICES = FFMPEG_VERBOSITY_MAP.keys()
-WEBGL_RENDERER_INFO: str = (
-    "The Electron frontend to Manim is hosted at "
-    "https://github.com/ManimCommunity/manim-renderer. After cloning and building it, "
-    "you can either start it prior to running Manim or specify the path to the "
-    "executable with the --webgl_renderer_path flag."
-)
 
 # Video qualities
 QUALITIES: dict[str, dict[str, str | int | None]] = {