Integer Default Parameters for Plotting Methods

The goal is to allow for this syntax;

emb.transform(Pca(2)).plot(x_axis=0, y_axis=1)

The reasoning is currently you have to write;

emb.transform(Pca(2)).plot(x_axis="pca_0", y_axis="pca_1")

This get’s annoying when you want to replace the Pca with Umap.

emb.transform(Umap(2)).plot(x_axis="umap_0", y_axis="umap_1")

What’s even nicer is that with proper default arguments in the future you can just run;

emb.transform(Umap(2)).plot()

[mkaze]

@koaning Let’s close this issue (since I think it’s fully resolved now) and continue the discussion for handling helper vectors in #199.

[mkaze]

@koaning Having implemented and merged this feature, now how do we want to go about handling those pca_0, pca_1, etc. embeddings? Do we want to entirely remove them, or make them optional (e.g. for backwards compatibility) by adding a separate argument in the __init__ method of corresponding transformer classes, e.g. def __init__(self, ..., add_indicators=False)? Or any other option?