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 alpaka to the devel branch as of 2022.06.21 #7986

Conversation

fwyzard
Copy link
Contributor

@fwyzard fwyzard commented Jul 8, 2022

Update alpaka to the HEAD of the devel branch as of 2022.06.21, corresponding to the commit 5a4691c82676176fd8b71e1f57bb809f8c75a095.

Major changes:

  • implement const buffers with the ViewConst adaptor;
  • ensure that tasks submitted to async CPU queues are completed even
    after the queue is destroyed;
  • use a callback thread instead of launching a new thread for each
    CUDA asynchronous host function.

Other changes:

  • added demangled type names;
  • introduce alpaka::math::constants.

Update the supported host compilers:

  • add support for gcc 12;
  • drop support for clang 5.

Various bug fixes.

Update alpaka to the HEAD of the devel branch as of 2022.06.21,
corresponding to the commit 5a4691c82676176fd8b71e1f57bb809f8c75a095.

Major changes:
  - implement const buffers with the ViewConst adaptor;
  - ensure that tasks submitted to async CPU queues are completed even
    after the queue is destroyed;
  - use a callback thread instead of launching a new thread for each
    CUDA asynchronous host function.

Other changes:
  - added demangled type names;
  - introduce alpaka::math::constants.

Update the supported host compilers:
  - add support for gcc 12;
  - drop support for clang 5.

Various bug fixes.
@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 8, 2022

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

@cmsbuild, @smuzaffar, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks.
@perrotta, @dpiparo, @qliphy, @rappoccio you are the release manager for this.
cms-bot commands are listed here

@fwyzard
Copy link
Contributor Author

fwyzard commented Jul 8, 2022

please test with cms-sw/cmssw#37716

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 9, 2022

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c55dcc/26078/summary.html
COMMIT: 04a6417
CMSSW: CMSSW_12_5_X_2022-07-07-2300/el8_amd64_gcc10
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmsdist/7986/26078/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c55dcc/26078/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c55dcc/26078/git-merge-result

Comparison Summary

Summary:

  • No significant changes to the logs found
  • Reco comparison results: 6 differences found in the comparisons
  • DQMHistoTests: Total files compared: 50
  • DQMHistoTests: Total histograms compared: 3654999
  • DQMHistoTests: Total failures: 14
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 3654963
  • DQMHistoTests: Total skipped: 22
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 8218.428 KiB( 49 files compared)
  • DQMHistoSizes: changed ( 11634.0,... ): 1369.738 KiB HLT/B2G
  • Checked 208 log files, 45 edm output root files, 50 DQM output files
  • TriggerResults: no differences found

@fwyzard fwyzard changed the title Update alpaka to the devel branch as of 2022.06.21 / 5a4691c8267 Update alpaka to the devel branch as of 2022.06.21 Jul 10, 2022
@fwyzard
Copy link
Contributor Author

fwyzard commented Jul 10, 2022

assign heterogeneous

@fwyzard
Copy link
Contributor Author

fwyzard commented Jul 10, 2022

+heterogeneous

@cmsbuild
Copy link
Contributor

New categories assigned: heterogeneous

@fwyzard,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks

@smuzaffar
Copy link
Contributor

+externals

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_12_5_X/master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy, @rappoccio (and backports should be raised in the release meeting by the corresponding L2)

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