Plots: Make matplotlib outputs available for more plugins

[sarah-segel]

Plots: Make matplotlib outputs available for more plugins

[KrissiHub]

It looks like he method for the matplotlib version for (e.g.) the importances plugin returns a plotly, not a matplotlib though. Am I seeing this right or can someone clarify?

[sarah-segel]

So by default, a plotly is returned. But when selecting matplotlib style, the method load_mpl_outputs() should be used instead of load_outputs():

DeepCAVE/deepcave/plugins/__init__.py

Line 506 in 1851126

if mpl_active:

I think currently the plugins parteto front and importance have that method. E.g. for the importance plugin:

DeepCAVE/deepcave/plugins/hyperparameter/importances.py

Line 291 in 1851126

def load_mpl_outputs(run, inputs, outputs):

Does that answer your question, or did I overlook something there?

[sarah-segel]

Nevertheless, there seem to be some problems with the existing matplotlib methods. E.g. they use seaborn style, but seaborn is not in the requirements - I will add that. For the pareto front, it seems like there is another problem as well, which needs to be checked at some point (it creates a plot but does not show the pareto front).

[KrissiHub]

I think the main part of my problem of understanding the method lies here

DeepCAVE/deepcave/plugins/hyperparameter/importances.py

Lines 284 to 288 in 1851126

	def get_mpl_output_layout(register):
	return html.Img(
	id=register("graph", "src"),
	className="img-fluid",
	)

If my understanding is correct, the returned object is a plotly object, but the methods name suggest, that it returns a matplotlib object.