The return value of plot methods

This is partly relevant to #127.

Currently, the following methods return a reference to the plot objects:

• plot_interactive_matrix
• plot_interactive
• plot_movement

However, the following methods either return None or self:

• plot_pixels
• plot_correlation
• plot_graph_layout
• plot from EmbeddingSet
• plot from Embedding

Isn’t it better that the methods in the latter group also return a reference to the plot object? This makes them all have similar interface, as well as gives the user the ability to tune the plot object further if desired (although, the methods in the second group use matplotlib and therefore the plots could be customized with plt.the_relevant_function(...) as well).

[koaning]

If I’m honest, I think the library doesn’t have that many users yet. Here’s the views for the relevant documentation page;

In 60 days maybe 100 people looked at it? That’s maybe two people a day? Assuming they don’t 100% fully read and instead just glance I’d say it may be not be the worst of worst things to happen. But we shouldn’t have breaking changes often.

As long as we update the docs immediately I think I can live with it.

[koaning]

I’m closing issues because ever since the project moved to my personal account it’s been more into maintenance mode than a “active work” mode.