New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add an example to create a custom converter for a NMF transformer #167
Conversation
:math:`M=(m_{ij})` may be a binary matrix where *i* is a user | ||
and *j* a product he bought. The prediction | ||
function depends on whether or not the user needs an | ||
recommandation for an existing user or a new user. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo here: "a recommendation"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"recommandation" typo still there.
docs/examples/plot_nmf.py
Outdated
from sklearn.decomposition import NMF | ||
import numpy as np | ||
import matplotlib.pyplot as plt | ||
import os |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a standard library import and should be placed before third-party imports. Sadly flake8 doesn't catch it, but you can run pylint locally on your code to see these violations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see the change: import os
should be at the very top.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolve the two remaining comments before merging.
:math:`M=(m_{ij})` may be a binary matrix where *i* is a user | ||
and *j* a product he bought. The prediction | ||
function depends on whether or not the user needs an | ||
recommandation for an existing user or a new user. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"recommandation" typo still there.
docs/examples/plot_nmf.py
Outdated
from sklearn.decomposition import NMF | ||
import numpy as np | ||
import matplotlib.pyplot as plt | ||
import os |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see the change: import os
should be at the very top.
No description provided.