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

fix: Howto material without spread #848

Merged
merged 3 commits into from
Jun 23, 2021

Conversation

Corentin-Allaire
Copy link
Contributor

Fix the Howto for material mapping. Include the fact that the Z0 and D0 sigma need to be set to 0 in the validation to be correctly compare to the Geantino scan.
Also modify the root script to take into account that the Geantino scan is flat in theta where the Validation is flat in eta.

@Corentin-Allaire Corentin-Allaire added Component - Documentation Affects the documentation Component - Examples Affects the Examples module Impact - Minor Nuissance bug and/or affects only a single module labels Jun 21, 2021
@Corentin-Allaire Corentin-Allaire added this to the next milestone Jun 21, 2021
@Corentin-Allaire Corentin-Allaire changed the title Fix: Howto material without spread fix: Howto material without spread Jun 21, 2021
@codecov
Copy link

codecov bot commented Jun 21, 2021

Codecov Report

Merging #848 (c2864c1) into main (d20e23c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #848   +/-   ##
=======================================
  Coverage   48.46%   48.46%           
=======================================
  Files         329      329           
  Lines       16974    16974           
  Branches     8028     8028           
=======================================
  Hits         8227     8227           
  Misses       3092     3092           
  Partials     5655     5655           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d20e23c...c2864c1. Read the comment docs.

Co-authored-by: Paul Gessinger <hello@paulgessinger.com>
@asalzburger asalzburger merged commit 54dcb61 into acts-project:main Jun 23, 2021
@paulgessinger paulgessinger modified the milestones: next, v0.9.1 Jun 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Documentation Affects the documentation Component - Examples Affects the Examples module Impact - Minor Nuissance bug and/or affects only a single module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants