Skip to content

Add c++23, gpu to nodeinfo #441

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

Open
jwokaty opened this issue Feb 20, 2025 · 2 comments
Open

Add c++23, gpu to nodeinfo #441

jwokaty opened this issue Feb 20, 2025 · 2 comments

Comments

@jwokaty
Copy link
Collaborator

jwokaty commented Feb 20, 2025

Should we add c++23 and gpu info if available to nodeinfo? For example https://bioconductor.org/checkResults/3.21/bioc-LATEST/kjohnson3-NodeInfo.html.

@hpages
Copy link
Contributor

hpages commented Feb 20, 2025

We only display C++ info up to standard 17 at the moment and there is standard 20 between 17 and 23 (see R/etc/Makeconf). So if we want to go all the way to 23 we should also display info for 20.

Also maybe we should start doing the same thing for the various C standards: 90, 99, 17, 23 so far.

GPU info is a different story. Displaying the output of nvidia-smi on Linux would be a good start (note that the command is not guaranteed to be available). Not sure about Windows or Mac. The GPU on Mac is probably the same for any given Mac model, I don't know. Anyways the GPU info conversation probably diserves a separate issue.

@hpages
Copy link
Contributor

hpages commented Feb 20, 2025

Note that R still uses -std=gnu++17 by default for the C++ compiler. The change they just made in R devel is to use
-std=gnu23 by default for the C compiler, but only if the compiler supports the 23 standard. The default C compiler on Ubuntu 24.04 does not support C23.

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

No branches or pull requests

2 participants