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

Fix CMakeList to include targets for CUDNN, NCCL #4

Closed
wants to merge 4 commits into from

Conversation

dmudiger
Copy link

No description provided.

@dmudiger
Copy link
Author

Minor changes to ensure right values for CUDNN, NCCL are passed.
Follow-up question (apologies for repurposing the PR, opening issues seem to be disabled) - it would help to include the ability to run with random data. Especially for performance measurements similar to results reported in the user guide (https://github.com/dmudiger/HugeCTR/blob/master/docs/hugectr_user_guide.md#hugectr)
(cc: @yuchenhao)

@minseokl
Copy link
Collaborator

Hi @dmudiger

  1. Thanks for your interest in HugeCTR. You can actually set the custom paths to cuDNN and NCCL by setting the environment variables CUDNN_DIR and NCCL_DIR before doing ./configuring. @zehuanw and I will update our REAME.md to make the build instructions clearer.
  2. By the way, could you briefly explain why you added build.sh? It seems like the file used in your computer. Did you commit it to illustrate how to configure and make the project?
  3. Your suggestion on the random data generation would be useful for the performance analysis and architectural research, but I need to have an internal discussion to decide if we should prioritize it. For now, I suggest to follow the intructions in this link.

If you have more questions and issues, please let me know anytime.

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.

3 participants