Basic logging added

[sauln]

Here is a first pass on #18. I have tested this in a notebook and a script.

I went with scikit-learn style logging (example can be seen here) instead of the gensim style as mentioned previously. If you'd rather have the python module be used, we can convert it. There might be issues getting it to work within the numba.njit decorated functions though.

Example usage:

import umap
from sklearn.datasets import load_digits

digits = load_digits()
embedding = umap.UMAP(verbose=True).fit_transform(digits.data)

Output:

UMAP(n_neighbors=15, n_components=2, metric='euclidean', gamma=1.0, n_edge_samples=None, alpha=1.0, init='spectral', spread=1.0, min_dist=0.1, a=None, b=None, random_state=None, metric_kwds={}, verbose=True)
Construct fuzzy simplicial set
	nn descent iteration  0  /  10
	nn descent iteration  1  /  10
	nn descent iteration  2  /  10
	nn descent iteration  3  /  10
Construct embedding
	embedding iteration  0  /  11000000
	embedding iteration  1100000  /  11000000
	embedding iteration  2200000  /  11000000
	embedding iteration  3300000  /  11000000
	embedding iteration  4400000  /  11000000
	embedding iteration  5500000  /  11000000
	embedding iteration  6600000  /  11000000
	embedding iteration  7700000  /  11000000
	embedding iteration  8800000  /  11000000
	embedding iteration  9900000  /  11000000

[pep8speaks]

Hello @sauln! Thanks for updating the PR.

• In the file umap/umap_.py, following are the PEP8 issues :

Line 155:80: E501 line too long (80 > 79 characters)
Line 350:80: E501 line too long (80 > 79 characters)
Line 449:80: E501 line too long (94 > 79 characters)
Line 573:5: E741 ambiguous variable name 'I'
Line 591:80: E501 line too long (80 > 79 characters)
Line 674:55: E203 whitespace before ':'
Line 709:80: E501 line too long (80 > 79 characters)
Line 710:9: E722 do not use bare except'
Line 831:80: E501 line too long (80 > 79 characters)

Comment last updated on November 16, 2017 at 07:35 Hours UTC

[sauln]

cd847e1 should fix most of those PEP8 issues.

[lmcinnes]

Thanks, this looks great! I can imagine we might want to do something more complex in the future, but its great to have something in place now.

At least a bunch of that PEP8 stuff is my fault.