Skip to content

Fix a code typo in the README file#5

Open
sangaline wants to merge 1 commit intomstrazar:masterfrom
sangaline:ews-fix-typo-in-readme
Open

Fix a code typo in the README file#5
sangaline wants to merge 1 commit intomstrazar:masterfrom
sangaline:ews-fix-typo-in-readme

Conversation

@sangaline
Copy link
Copy Markdown

The README currently lists an example of

    import numpy as np
    from ionmf.factorization.onmf import onmf
    X = np.random.rand(10, 10)
    W, H = onmf(X, k=5, alpha=1.0)

but this code doesn't work because k isn't a valid argument to onmf. This PR replaces k with rank which is the actual parameter for the number of dimensions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant