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

[EMCAL-630] Fix assignment of energy and time #5225

Merged
merged 1 commit into from Jan 19, 2021

Conversation

mfasDa
Copy link
Collaborator

@mfasDa mfasDa commented Jan 19, 2021

  • Energy must be protected only against negative energies
  • Time from the raw fit also only protected against negative entries
  • Cell converter needs to store amplitued and time from the
    raw fit results.

- Energy must be protected only against negative energies
- Time from the raw fit also only protected against negative entries
- Cell converter needs to store amplitued and time from the
  raw fit results.
Copy link
Collaborator

@shahor02 shahor02 left a comment

Choose a reason for hiding this comment

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

trivial fix, merging

@shahor02 shahor02 merged commit 9cc9887 into AliceO2Group:dev Jan 19, 2021
@mfasDa mfasDa deleted the EMCAL-630 branch January 19, 2021 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants