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

Consumes jet met #575

Merged
merged 7 commits into from Aug 21, 2013
Merged

Consumes jet met #575

merged 7 commits into from Aug 21, 2013

Conversation

perrotta
Copy link
Contributor

Migrate to consumes/getByToken in all modules
Add also fillDescriptions() when missing

@ktf
Copy link
Contributor

ktf commented Aug 20, 2013

@perrotta @Martin-Grunewald can you please sign (still using the topic collector for the moment).

@Martin-Grunewald can you confirm you receive this email?

@ktf
Copy link
Contributor

ktf commented Aug 20, 2013

@nclopezo can you please run the automated tests?

@cmsbuild
Copy link
Contributor

The following categories have been signed by gruenew (a.k.a. @Martin-Grunewald on GitHub): HLT

@cms-git-hlt, @cms-git-fastsim

@Martin-Grunewald
Copy link
Contributor

Hi,

I signed the request - (and received the e-mail - due to the explicit cc?).

Best regards

On Tue, 20 Aug 2013, Giulio Eulisse wrote:

@perrotta @Martin-Grunewald can you please sign (still using the topic collector for the moment).

@Martin-Grunewald can you confirm you receive this email?


Reply to this email directly or view it on GitHub:
#575 (comment)

Martin

Martin W. Gruenewald e-mail: Martin.Grunewald@cern.ch
http://cern.ch/Martin.Grunewald

@ghost ghost assigned nclopezo Aug 21, 2013
@nclopezo
Copy link
Contributor

Hi,

I took these changes on top of CMSSW_7_0_X_2013-08-21-0200, I ran the unit tests and RelVals, all tests passed.

You can see the logs here:

https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc472/194/consoleFull

cmsbuild added a commit that referenced this pull request Aug 21, 2013
@cmsbuild cmsbuild merged commit 8bb2e4a into cms-sw:CMSSW_7_0_X Aug 21, 2013
@perrotta perrotta deleted the consumesJetMET branch September 23, 2013 14:35
jbsauvan pushed a commit to jbsauvan/cmssw that referenced this pull request Jan 26, 2016
cmsbuild pushed a commit that referenced this pull request Nov 27, 2020
Remove SiPixelDigiHeterogeneousConverter as obsolete, should have been removed as part of cms-patatrack#100.

Address review comments for SiPixelClustersCUDA:
  - remove commented out default constructor and private: from DeviceConstView;
    this is perhaps the best compromise between non-default constructors not
    being preferred for device allocations, and the use case in
    SiPixelRecHitSoAFromLegacy (for the expected life time of this class)
  - remove const getters with c_ prefix
  - improve constructor parameter name
  - use more initializer list
  - initialize nClusters_h

Address review comments for SiPixelDigiErrorsCUDA:
  - use type alias
  - remove const getters with c_ prefix and other unnecessary methods
  - use more initializer list

Address review comments for SiPixelDigisCUDA:
  - remove const getters with c_ prefix and other unnecessary methods
  - remove commented out default constructor and private: from DeviceConstView
  - add comments for remaining SiPixelDigisCUDA member arrays

Move PixelErrorsCompact and SiPixelDigiErrorsSoa to DataFormats/SiPixelRawData, rename classes

Address review comments for SiPixelErrorsSoA
  - remove redundant assert
  - move constructor inline

Address review comments for SiPixelDigisSoA
  - remove redundant assert
  - add comments

Enable if constexpr also for CUDA in TrackingRecHit2DHeterogeneous

Move dictionary of HostProduct<unsigned int[]> to CUDADataFormats/Common
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

5 participants