ICSM / ampere

A tool to fit the SED and spectra of dusty objects to constrain, among other things, the dust properties and mineralogy
6 stars 2 forks source link

The following plots need to be adapted for large numbers of parameters: corner, lnprob and walkers #61

Open SaschaZeegers opened 2 years ago

SaschaZeegers commented 2 years ago

In case there are many parameters some of the plots become hard to read, especially corner, lnprob and walkers. A solution might be to group the parameters and make multiple sub plots.

pscicluna commented 2 years ago

There are 3 things we can do.

  1. We can make the walkers and corner plots automatically calculate a figure size to give each panel the same amount of space, so that the figure expands. However, the user would have to zoom in and we would have to make all output vector graphics (eps or pdf)) to make zooming smooth.
  2. Enable user-specified choice of parameters to plot. Corner and arviz (used under the hood) already do this, so we just need to expose those same mechanisms
  3. Make a choice ourselves about plotting only some parameters, or plotting groups of parameters. This might result in large numbers of corner plots, however, as we want to plot all combinations eventually

Some combination of all 3 seems like the best course, by doing 2 and using the resulting code to set some sensible defaults for 3, while also implementing automatic resizing.

I suggest making the first steps be (simultaneously) implementing auto-resizing and user-specified choice of parameters to plot? @SaschaZeegers are you interested in taking on the automatic resizing?

SaschaZeegers commented 2 years ago

Hi Peter,

Yes, I can take a look at the automatic resizing. It might take a few weeks before I can start working on it.