Skip to content
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

docs: fix simple typo, imcompatible -> incompatible #190

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
docs: fix simple typo, imcompatible -> incompatible
There is a small typo in tests/factorization/test_explicit.py.

Should read `incompatible` rather than `imcompatible`.

Signed-off-by: Tim Gates <[email protected]>
timgates42 committed Jul 18, 2022

Verified

This commit was signed with the committer’s verified signature.
timgates42 Tim Gates
commit c62a6190cc7c3e20ab27cfe171ce4323fcec3fab
2 changes: 1 addition & 1 deletion tests/factorization/test_explicit.py
Original file line number Diff line number Diff line change
@@ -97,7 +97,7 @@ def test_check_input():
use_cuda=CUDA)
model.fit(train)

# Modify data to make imcompatible with original model.
# Modify data to make incompatible with original model.
train.user_ids[0] = train.user_ids.max() + 1
with pytest.raises(ValueError):
model.fit(train)