-
Notifications
You must be signed in to change notification settings - Fork 90
Intel(R) oneAPI Collective Communications Library (oneCCL) 2021.15.1 #167
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
Intel(R) oneAPI Collective Communications Library (oneCCL) 2021.15.1 #167
Conversation
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.
we probably want to keep the deprecation notice
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.
done
Added * to NCCL to refer the reader to the new Notices and Disclaimers section (taken from our release notes) which addresses the trademark owership for NCCL. Used improvements suggested by Nikita.
I learned that I need to use the * only on the first instance of the trademarked name. Removed the second *.
Co-authored-by: Ranu Kundu <[email protected]>
Co-authored-by: Ranu Kundu <[email protected]>
Co-authored-by: Ranu Kundu <[email protected]>
Co-authored-by: Ranu Kundu <[email protected]>
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.
The man changes seem strange but they are aligned with the internal branch.
Looks good.
This PR aligns the contents of the repository with the Intel(R) oneAPI Collective Communications Library 2021.15.1 release.