fix: Do nothing in update_colour_mapping if model_ids is empty

app.py

@@ -353,18 +353,15 @@ def main() -> None:
             "Alexandra Institute</a>.</center>"
         )
 
+        # Update plot when anything changes
         language_names_dropdown.change(
             fn=partial(update_model_ids_dropdown, results_dfs=results_dfs),
             inputs=[language_names_dropdown, model_ids_dropdown],
             outputs=model_ids_dropdown,
-        ).then(fn=update_colour_mapping, inputs=model_ids_dropdown)
-
-        # Update plot when anything changes
-        update_plot_kwargs = dict(
-            fn=partial(
-                produce_radial_plot,
-                results_dfs=results_dfs,
-            ),
+        ).then(
+            fn=update_colour_mapping, inputs=model_ids_dropdown
+        ).then(
+            fn=partial(produce_radial_plot, results_dfs=results_dfs),
             inputs=[
                 model_ids_dropdown,
                 language_names_dropdown,
@@ -375,24 +372,76 @@ def main() -> None:
             ],
             outputs=plot,
         )
-        language_names_dropdown.change(
-            fn=update_colour_mapping, inputs=model_ids_dropdown
-        ).then(**update_plot_kwargs)
         model_ids_dropdown.change(
             fn=update_colour_mapping, inputs=model_ids_dropdown
-        ).then(**update_plot_kwargs)
+        ).then(
+            fn=partial(produce_radial_plot, results_dfs=results_dfs),
+            inputs=[
+                model_ids_dropdown,
+                language_names_dropdown,
+                use_rank_score_checkbox,
+                show_scale_checkbox,
+                plot_width_slider,
+                plot_height_slider,
+            ],
+            outputs=plot,
+        )
         use_rank_score_checkbox.change(
             fn=update_colour_mapping, inputs=model_ids_dropdown
-        ).then(**update_plot_kwargs)
+        ).then(
+            fn=partial(produce_radial_plot, results_dfs=results_dfs),
+            inputs=[
+                model_ids_dropdown,
+                language_names_dropdown,
+                use_rank_score_checkbox,
+                show_scale_checkbox,
+                plot_width_slider,
+                plot_height_slider,
+            ],
+            outputs=plot,
+        )
         show_scale_checkbox.change(
             fn=update_colour_mapping, inputs=model_ids_dropdown
-        ).then(**update_plot_kwargs)
+        ).then(
+            fn=partial(produce_radial_plot, results_dfs=results_dfs),
+            inputs=[
+                model_ids_dropdown,
+                language_names_dropdown,
+                use_rank_score_checkbox,
+                show_scale_checkbox,
+                plot_width_slider,
+                plot_height_slider,
+            ],
+            outputs=plot,
+        )
         plot_width_slider.change(
             fn=update_colour_mapping, inputs=model_ids_dropdown
-        ).then(**update_plot_kwargs)
+        ).then(
+            fn=partial(produce_radial_plot, results_dfs=results_dfs),
+            inputs=[
+                model_ids_dropdown,
+                language_names_dropdown,
+                use_rank_score_checkbox,
+                show_scale_checkbox,
+                plot_width_slider,
+                plot_height_slider,
+            ],
+            outputs=plot,
+        )
         plot_height_slider.change(
             fn=update_colour_mapping, inputs=model_ids_dropdown
-        ).then(**update_plot_kwargs)
+        ).then(
+            fn=partial(produce_radial_plot, results_dfs=results_dfs),
+            inputs=[
+                model_ids_dropdown,
+                language_names_dropdown,
+                use_rank_score_checkbox,
+                show_scale_checkbox,
+                plot_width_slider,
+                plot_height_slider,
+            ],
+            outputs=plot,
+        )
 
     demo.launch()
 
@@ -740,6 +789,9 @@ def update_colour_mapping(model_ids: list[str]) -> None:
         model_ids:
             The model ids to update the colour
     """
+    if not model_ids:
+        return
+
     global colour_mapping
     global seed
     seed += 1