xiuminglib icon indicating copy to clipboard operation
xiuminglib copied to clipboard

Eliminate `vis.general`

Open xiumingzhang opened this issue 6 years ago • 3 comments

... by renaming it to vis.plot and factoring out those that don't fit into the new name, such as vis.general._save.

xiumingzhang avatar Oct 11 '19 18:10 xiumingzhang

Another idea: having a new vis.wrap module that contains separate wrappers for pyplot.scatter, pyplot.line, etc. The current universal wrapper is too ambitious, with lots of corner cases.

xiumingzhang avatar May 09 '20 19:05 xiumingzhang

ok, going with the 2nd option, but naming it vis.plot instead.

Start migrating functions from vis.general to vis.plot.

xiumingzhang avatar Sep 29 '20 03:09 xiumingzhang

Makes more sense to have specialized functions, such as vis.line, vis.scatter, etc. Otherwise, this kind of general wrapper is as hard to use as the original matplotlib API...

xiumingzhang avatar May 10 '21 16:05 xiumingzhang