-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[DOC] Update ssd doc to avoid confusion. #3677
Conversation
# change target argument to 'opencl -device=intel_graphics' | ||
# To get best inference performance on Intel graphics, | ||
# change target argument to :code:`opencl -device=intel_graphics`, | ||
# except for Mac, target needs to be set to `opencl` only. |
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.
not sure whether it is clear for native speaker ... "except for Mac, target needs to be set to opencl only." means set "opencl" for mac, or for non-mac?
shall we also mention intel extension on mac?
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, lemme know how it looks now.
Thanks! |
* intel graphics conv2d bugs fixed for inception_v3 * intel conv2d api updated, nn input size 4 condition added * review addressed * move conv_tags to attributes * ssd doc updated * address comment
* intel graphics conv2d bugs fixed for inception_v3 * intel conv2d api updated, nn input size 4 condition added * review addressed * move conv_tags to attributes * ssd doc updated * address comment
@yzhliu Could you review?