-
Notifications
You must be signed in to change notification settings - Fork 138
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
[Feat]combine cpu and gpu wheel #456
Conversation
36ea21a
to
55a594e
Compare
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.
LGTM
README.md
Outdated
@@ -76,7 +76,7 @@ run the following: | |||
pip install tensorflow-recommenders-addons | |||
``` | |||
|
|||
By default, CPU version will be installed. To install GPU version, run the following: | |||
Before the release of combined CPU and GPU version, to install GPU version, run the following: |
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.
Better to explain more clearly, for example, the specific version(0.7.x) after which we make this change.
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.
updated
Hey @MoFHeka pls review it when you feel convenient, basically, it's LGTM. |
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.
LGTM
Description
Brief Description of the PR:
combine cpu and gpu wheel so that you only need pip install tensorflow-recommenders-addons
Fixes # (issue)
Type of change
Checklist:
How Has This Been Tested?
If you're adding a bugfix or new feature please describe the tests that you ran to verify your changes:
*