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

temporary PR to be able to make comments inline #61

Open
wants to merge 55 commits into
base: CMSSW_6_2_0_SLHC14/TT-GMT-analysis-ttiRedigiJun14ana-gem-sw
Choose a base branch
from
Open

temporary PR to be able to make comments inline #61

wants to merge 55 commits into from

Conversation

slava77
Copy link
Owner

@slava77 slava77 commented Jan 23, 2015

No description provided.


};

struct L1TkMuonCand {
Copy link
Owner Author

Choose a reason for hiding this comment

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

This is the input L1 muon type, rename to L1MuonCand

@slava77
Copy link
Owner Author

slava77 commented Jan 23, 2015

The goal of conversions from c++ to C-structs was to have no complex C++ class types used between

https://github.com/slava77/cmssw/blob/CMSSW_6_2_0_SLHC14/TT-GMT-analysis-ttiRedigiJun14ana-gem-sw/SLHCUpgradeSimulations/L1TrackTrigger/plugins/L1TkMuonFromExtendedProducer.cc#L146
and
https://github.com/slava77/cmssw/blob/CMSSW_6_2_0_SLHC14/TT-GMT-analysis-ttiRedigiJun14ana-gem-sw/SLHCUpgradeSimulations/L1TrackTrigger/plugins/L1TkMuonFromExtendedProducer.cc#L271

Currently, it's still a merge of the two. (I know that it's a work in progress, just wanted to make it a bit more clear where the C++ inputs -> simple-C-struct conversion is supposed to go.)

PropState pstate = propagateToGMT(l1tk);
if (!pstate.valid) continue;

L1Tk[il1tk].matchPt = pstate.pt;
Copy link
Owner Author

Choose a reason for hiding this comment

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

better don't skip, just fill the bad default values (should come for a failed case).

slava77 pushed a commit that referenced this pull request Feb 17, 2017
slava77 pushed a commit that referenced this pull request Nov 6, 2017
slava77 pushed a commit that referenced this pull request Nov 29, 2017
update b-tag discriminators
slava77 pushed a commit that referenced this pull request Jul 1, 2019
…PUCuda (#61)

  - add some sanity checks
  - add a minimal documentation of the heterogeneous framework
@sergueibaranov sergueibaranov deleted the CMSSW_6_2_0_SLHC14/TT-GMT-analysis-ttiRedigiJun14ana-gem-sw branch September 28, 2020 06:58
slava77 pushed a commit that referenced this pull request Mar 9, 2021
* Moved class matrix to its own header file, and renamed to Array2D

* Moved class matrix to its own header file, and renamed to Array2D
slava77 pushed a commit that referenced this pull request Oct 9, 2021
slava77 pushed a commit that referenced this pull request Oct 29, 2021
Seed partition function with TID detection
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.

None yet

1 participant