Closed vkrajnak closed 3 years ago
Maybe work on redefinition of colourmap parameters to achieve this.
@VikJGG plots do something like that and when overlaid they look great (pics below). However, is this colour scale tweaking manual?
Just a thought: One possible approach would be to take the LD values distribution and define vmin
and vmax
passed to plt.contourf
as a function of the mean and std of LD values. I did something like this for the function draw_lagrangrian_descriptors
, defining a new variable called colormap_mode
that implements these changes of vmin/vmax if called with value 2.
Maybe it would be enough to redefine the plotting function to accept LD_forward and LD_backward optionally and depending on which ones are provided, it will draw forward, backward or both on top of each other. This should also be relatively straight-forward.
Maybe it would be enough to redefine the plotting function to accept LD_forward and LD_backward optionally and depending on which ones are provided, it will draw forward, backward or both on top of each other. This should also be relatively straight-forward.
Nice, can you do this @vkrajnak , please? I think this will enhance the graphical output for the EXAMPLES notebook, both for the book and the package.
I can put it on the waiting list
NOTE: Laplacian filter. Thanks, @VikJGG
For filters, you opened issue #11. Let's keep them out of this issue
Maybe work on redefinition of colourmap parameters to achieve this.
@VikJGG plots do something like that and when overlaid they look great (pics below). However, is this colour scale tweaking manual?
Just a thought: One possible approach would be to take the LD values distribution and define
vmin
andvmax
passed toplt.contourf
as a function of the mean and std of LD values. I did something like this for the functiondraw_lagrangrian_descriptors
, defining a new variable calledcolormap_mode
that implements these changes of vmin/vmax if called with value 2.
IMPORTANT NOTE
For filters, you opened issue #11. Let's keep them out of this issue
Gradient- and Laplacian-based filters should be defined under a common umbrella. I will stick to the other open issue though atm.
Addressed in pull request #30
Red for unstable, blue for stable. Will require to give draw_lagrangian_descriptor forward and backward LDs separately instead of sum.