krober10nd / SeismicMesh

2D/3D serial and parallel triangular mesh generation tool for finite element methods.

https://seismicmesh.readthedocs.io/

GNU General Public License v3.0

129 stars 33 forks source link

switching to svg for figures #176

Closed krober10nd closed 3 years ago

krober10nd commented 3 years ago

168

nschloe commented 3 years ago

The ylim[0] should always be 0 in the figures, and for the cell quality ylim[1] should always be 1.

nschloe commented 3 years ago

Also, some plots still have ylabels.

krober10nd commented 3 years ago

Also, some plots still have ylabels.

Yes, the second plot does by design. I prefer it.

nschloe commented 3 years ago

Yes, the second plot does by design. I prefer it.

title and ylabel virtually say the same thing (Speedup T1/TC vs Parallel Speedup, Mesh generation rate (vertices/sec) vs Mesh generation rate). One of the two is redundant. I'd remove the one that's harder the read, and merge the units into the one that remains.