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

Add Nsight Compute CLI python libraries #4630

Conversation

fwyzard
Copy link
Contributor

@fwyzard fwyzard commented Jan 10, 2019

Properly package the Nsight Compute command line client and python libraries.

Properly package the Nsight Compute command line client and python libraries.
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @fwyzard (Andrea Bocci) for branch IB/CMSSW_10_5_X/gcc700.

@cmsbuild, @smuzaffar, @gudrutis, @mrodozov can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.

@fwyzard
Copy link
Contributor Author

fwyzard commented Jan 10, 2019

@smuzaffar do you think this should work ?

The idea is to have in the PATH the file nv-nsight-cu-cli, which internally will launch $CUDA_BASE/NsightCompute-1.0/target/linux-desktop-glibc_2_11_3-glx-x64/nv-nsight-cu-cli .

If it works, do you think it would make sense to merge the cuda-gdb wrapper package here as well ?

@smuzaffar
Copy link
Contributor

@fwyzard , this should work. By the way, nv-nsight-cu-cli is already in PATH, so why you want to add wrapper?

Looks like nv-nsight-cu-cli is broken for slc6 (due to GLIBC version mismatch)

nv-nsight-cu-cli -v
nv-nsight-cu-cli: /lib64/libc.so.6: version `GLIBC_2.14' not found (required by nv-nsight-cu-cli)

@fwyzard
Copy link
Contributor Author

fwyzard commented Jan 11, 2019

By the way, nv-nsight-cu-cli is already in PATH, so why you want to add wrapper?

I put it in the PATH moving it and its support libraries from $CUDA_BASE/NsightCompute-1.0/target/linux-desktop-glibc_2_11_3-glx-x64/nv-nsight-cu-cli to $CUDA_BASE/bin .
However I just discovered that it looks for some support files in ../../ProfileSectionTemplates by default, which then gets broken.

@fwyzard
Copy link
Contributor Author

fwyzard commented Jan 11, 2019

Looks like nv-nsight-cu-cli is broken for slc6 (due to GLIBC version mismatch)

Not much we can do - RHEL 6 and RHEL 7 use the same installer - apart from a bug report to NVIDIA.

@fwyzard
Copy link
Contributor Author

fwyzard commented Jan 11, 2019

@cmsbuild, please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 11, 2019

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/32532/console

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

Comparison job queued.

@cmsbuild
Copy link
Contributor

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4630/32532/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 2 differences found in the comparisons
  • DQMHistoTests: Total files compared: 33
  • DQMHistoTests: Total histograms compared: 3153717
  • DQMHistoTests: Total failures: 13
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 3153500
  • DQMHistoTests: Total skipped: 204
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 32 files compared)
  • Checked 137 log files, 14 edm output root files, 33 DQM output files

@fwyzard
Copy link
Contributor Author

fwyzard commented Jan 12, 2019

@smuzaffar, is it possible to test the external built by Jenkins interactively ?

@fwyzard
Copy link
Contributor Author

fwyzard commented Jan 15, 2019

OK, from a local build it seems to work as intended.

@smuzaffar smuzaffar merged commit b3a0535 into cms-sw:IB/CMSSW_10_5_X/gcc700 Jan 16, 2019
@smuzaffar
Copy link
Contributor

@fwyzard , no it is not possible to test it interactively from jenkins. Once the jenkins job is done then the build area is cleaned up.
We can upload the RPMS to e.g. cms-PR<NUM> repository for any extra tests if needed.

@fwyzard fwyzard deleted the IB/CMSSW_10_5_X/gcc700_NsightComputeCLI branch May 21, 2019 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants