porespy icon indicating copy to clipboard operation
porespy copied to clipboard

Visualization helper functions are inconsistent

Open ma-sadeghi opened this issue 1 year ago • 1 comments

  • Some of them return both the figure and axis handles, i.e., fig, ax, and some return just return the figure handle. I think they should all return both.
  • Currently, they don't accept axis handle as an argument and instead they always create a new axis. In my experience, it's more common that plotting functions accept the axis handle for more granular control (like if you want to overlay them on an already existing axis, etc. Of course, it's still possible if you do the other plot afterwards)

ma-sadeghi avatar Jul 26 '23 18:07 ma-sadeghi

This is related to #849

jgostick avatar Jun 24 '24 15:06 jgostick