-
Notifications
You must be signed in to change notification settings - Fork 50
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 multi-gpu github action tests using horovod #1009
Conversation
Documentation preview |
@jperez999 fyi. do you mind reviewing? thanks. |
branch=main | ||
if [[ $ref_type == "tag"* ]] | ||
then | ||
git -c protocol.version=2 fetch --no-tags --prune --progress --no-recurse-submodules --depth=1 origin +refs/heads/release*:refs/remotes/origin/release* |
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.
Just curious, asking for my own edification: what's different in protocol.version=2
?
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 was copied from other places that have this pattern. now available in the branch-name action here. The reason for the protocol.version=2
was to match what the GitHub actions/checkout does in the fetch. It may no longer be strictly necessary since the default protocol is v2 with recent versions of git.
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.
It looks like v2 enables more commands to be sent over a single connection, among other things: https://git-scm.com/docs/protocol-v2
Goals ⚽
This PR adds a GitHub Actions workflow for multi-GPU tests.
Implementation Details 🚧
Testing Details 🔍
GitHub Actions workflow:
horovod (2GPU) / gpu-ci