Skip to content
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

Create WDPM_mulGPU.c #66

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leogean3
Copy link
Collaborator

@leogean3 leogean3 commented May 2, 2021

Upload the multiple-GPU code of WDPM

Upload the multiple-GPU code of WDPM
@KevinShook
Copy link
Contributor

Thanks for uploading.
I'm not sure that this should go in the master branch yet.
We'll have to add a section to the manual, which will push back the publication of
the paper.
Could you make a new branch for it, so that we can work on the manual and
anything else that needs to be updated?

@leogean3
Copy link
Collaborator Author

leogean3 commented May 2, 2021

Ah sorry, I originally want to upload the code to the "/src" folder... But sure I will make a new branch for this. By the way, is the current paper (the one that is under reviewing) needs to include the multiple-GPU implementation?

@KevinShook
Copy link
Contributor

Thanks.
There's no mention of the multiple-gpu version in the paper.
I think that Ray is thinking it would be a separate paper as it's all
your MSc work.

@leogean3
Copy link
Collaborator Author

Hi Kevin, seems that I do not have permission to create a new branch. The multi-GPU code is in this pull request. If necessary, could you please create a new branch, then I put the code there?

@KevinShook
Copy link
Contributor

KevinShook commented Sep 20, 2021 via email

@leogean3
Copy link
Collaborator Author

Thanks:)

@KevinShook
Copy link
Contributor

I've sent you an invitation to be an outside collaborator.
Once you accept, I will give you the required permission

@leogean3
Copy link
Collaborator Author

Thanks very much, I've accepted the invitation:)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants