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.

Sign up for GitHub

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

Jump to bottom

EGamma crystals #17

Closed
andreypz opened this issue Oct 22, 2013 · 3 comments
Closed

EGamma crystals #17

andreypz opened this issue Oct 22, 2013 · 3 comments
Assignees
@andreypz
Labels
bug enhancement request
Milestone
When it is not sh...

Comments

@andreypz
Copy link
Contributor

Need to address the issue of saving Ecal crystals. Currently, for the Photons, there are up to 100 (!) crystals saved, each contains 10 variables.
This accounts for 3% of the ntuple size of double electron sample.
@ghost ghost assigned andreypz Oct 22, 2013
@pollackscience
Copy link
Member

Physical size is fairly unimportant, memory usage is. Work on that instead

On Oct 22, 2013, at 7:30 PM, Andrey Pozdnyakov notifications@github.com wrote:

Need to address the issue of saving Ecal crystals. Currently, for the Photons, there are up to 100 (!) crystals saved, each contains 10 variables.
This accounts for 3% of the ntuple size of double electron sample.

—
Reply to this email directly or view it on GitHub.

@pollackscience
Copy link
Member

Actual bug: Crystals are not properly initialized, will fix

@pollackscience
Copy link
Member

fixed, devPollack_v9X

@andreypz andreypz mentioned this issue Jul 13, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andreypz andreypz

Labels
bug enhancement request
Projects
None yet
Milestone
When it is not shameful to show the c...
Development

No branches or pull requests
2 participants
@andreypz @pollackscience