gloo icon indicating copy to clipboard operation
gloo copied to clipboard

Rename RCCL_LIBRARY to RCCL_LIBRARY_PATH

Open jithunnair-amd opened this issue 5 years ago • 14 comments

to avoid collision with variable in RCCL cmake file. This should fix the error about not finding "-lrccl" in https://github.com/pytorch/pytorch/pull/31341 (now refiled as https://github.com/pytorch/pytorch/pull/34683)

jithunnair-amd avatar Dec 17 '19 00:12 jithunnair-amd

Hi jithunnair-amd! Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file.In order for us to review and merge your code, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.

If you have received this in error or have any questions, please contact us at [email protected]. Thanks!

facebook-github-bot avatar Dec 17 '19 00:12 facebook-github-bot

@pietern Any guidance on the above CLA stuff?

jithunnair-amd avatar Dec 18 '19 00:12 jithunnair-amd

@jithunnair-amd Checking in internally on how to proceed.

pietern avatar Dec 19 '19 09:12 pietern

@pietern Any update on this?

jithunnair-amd avatar Jan 02 '20 18:01 jithunnair-amd

@jithunnair-amd We still require a CLA for repo's outside the pytorch org. I think AMD already has one, so it should be possible to whitelist. Can you check on your side?

pietern avatar Jan 06 '20 09:01 pietern

@pietern I signed the CLA, so this should be good to commit.

jithunnair-amd avatar Mar 12 '20 21:03 jithunnair-amd

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

facebook-github-bot avatar Mar 12 '20 21:03 facebook-github-bot

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

facebook-github-bot avatar Mar 12 '20 22:03 facebook-github-bot

@pietern Any concerns with merging this in?

jithunnair-amd avatar Apr 15 '20 20:04 jithunnair-amd

@xw285cornell are you able to land this?

jeffdaily avatar Nov 24 '20 18:11 jeffdaily

This diff is very old, could you rebase and trigger the tests again?

xw285cornell avatar Nov 24 '20 19:11 xw285cornell

@xw285cornell Done

jithunnair-amd avatar Nov 24 '20 20:11 jithunnair-amd

@xw285cornell Pinging post-holiday break

jithunnair-amd avatar Nov 30 '20 18:11 jithunnair-amd

@xw285cornell are we good now? Thanks.

jeffdaily avatar Dec 02 '20 18:12 jeffdaily