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

Software compensation energy correction externalized #13

Merged
merged 3 commits into from
Oct 14, 2017
Merged

Software compensation energy correction externalized #13

merged 3 commits into from
Oct 14, 2017

Conversation

rete
Copy link
Contributor

@rete rete commented Sep 28, 2017

BEGINRELEASENOTES

  • Added separate registration function for software compensation energy correction
  • Software compensation parameters added in processor parameters
    • SoftwareCompensationWeights : the sc weights
    • SoftwareCompensationEnergyDensityBins : the energy density bins
    • FinalEnergyDensityBin : the final energy density value

ENDRELEASENOTES

… correction. Software compensation parameters added in processor parameters : weights, bins and final energy density value.
@rete
Copy link
Contributor Author

rete commented Sep 28, 2017

This PR is also related to PR19 in LCContent package : PandoraPFA/LCContent#19
Travis will surely fails until the LCContent PR is merged

rete added 2 commits October 10, 2017 16:42
…gy, m_minCleanHitEnergyFraction and m_minCleanCorrectedHitEnergy for software compensation in main settings
@rete
Copy link
Contributor Author

rete commented Oct 13, 2017

This builds on my computer without problems. Should we wait for the nightly build with the new LCContent ?

@andresailer
Copy link
Contributor

restarted the build with new nightly, passing now

@andresailer andresailer merged commit 1781bb6 into iLCSoft:master Oct 14, 2017
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.

2 participants