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

update cray documentation #2022

Merged
merged 3 commits into from
Nov 3, 2022
Merged

update cray documentation #2022

merged 3 commits into from
Nov 3, 2022

Conversation

boeschf
Copy link
Contributor

@boeschf boeschf commented Oct 27, 2022

  • updated documentation about cray systems
  • added comments about cuda target

Copy link
Contributor

@brenthuisman brenthuisman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor remarks.


.. code-block:: bash
This step is no longer required beginning with the Cray PE 10.06 release, where
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If no longer required, then remove.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Depends on the Cray PE version. What do you suggest instead?

Copy link
Contributor

@brenthuisman brenthuisman Nov 2, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Cray PE" is not defined, so it may not be obvious what it refers to our how a user controls it. I imagine it's an env with a set of GCC builds etc. one can load? Or is it done system wide and has the user no choice?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cray PE= Cray Programming Environment. Yes it comes with a bunch of tools: compilers, MPI, scientific libraries, but it also sets a bunch of environment variables etc. It usually the minimum you have on a system provided by Cray (HPE).

@boeschf boeschf requested a review from brenthuisman November 1, 2022 08:20
Co-authored-by: Brent Huisman <brenthuisman@users.noreply.github.com>
@brenthuisman brenthuisman merged commit aba5799 into arbor-sim:master Nov 3, 2022
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