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

refactor: Make Fatras Particle and simulation surface aware #2546

Merged

Conversation

andiwand
Copy link
Contributor

@andiwand andiwand commented Oct 16, 2023

Currently our Fatras simulation does not remember the surface particles were created on. This is a problem for the navigation as we cannot know if a close by surface was already encountered or not.

I propose to add a reference surface to the Fatras Particle which can be carried through in the propagation and then be used as a starting point for a new propagation.

We will also have to decide if a photon conversion on a sensitive surface should result in a hit or not. But this should be done in a separate PR if possible.

blocked by

@andiwand andiwand added this to the next milestone Oct 16, 2023
@github-actions github-actions bot added Component - Fatras Affects the Fatras module Event Data Model labels Oct 16, 2023
@github-actions github-actions bot added the Infrastructure Changes to build tools, continous integration, ... label Oct 16, 2023
@andiwand andiwand added the 🛑 blocked This item is blocked by another item label Oct 16, 2023
@codecov
Copy link

codecov bot commented Oct 16, 2023

Codecov Report

Merging #2546 (1ca7d64) into main (be4433c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2546   +/-   ##
=======================================
  Coverage   49.64%   49.64%           
=======================================
  Files         471      471           
  Lines       26632    26632           
  Branches    12237    12237           
=======================================
  Hits        13222    13222           
  Misses       4742     4742           
  Partials     8668     8668           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions github-actions bot removed the Infrastructure Changes to build tools, continous integration, ... label Oct 17, 2023
@andiwand andiwand added 🚧 WIP Work-in-progress 🛑 blocked This item is blocked by another item and removed 🛑 blocked This item is blocked by another item 🚧 WIP Work-in-progress labels Oct 17, 2023
@github-actions github-actions bot added Infrastructure Changes to build tools, continous integration, ... Component - Examples Affects the Examples module Changes Performance labels Oct 24, 2023
@andiwand andiwand removed the 🛑 blocked This item is blocked by another item label Oct 24, 2023
benjaminhuth
benjaminhuth previously approved these changes Oct 24, 2023
@kodiakhq kodiakhq bot merged commit 630fc20 into acts-project:main Oct 25, 2023
54 checks passed
@andiwand andiwand deleted the refactor-make-fatras-particle-surface-aware branch October 25, 2023 06:10
kodiakhq bot pushed a commit that referenced this pull request Oct 25, 2023
I encountered looping navigation between boundary surfaces as we do the lookup with overstep limit.

I believe we do not want to look for any boundaries that are behind us? But I might overlook something

Pulled this out of #2518

blocked by
- #2545
- #2546
@paulgessinger paulgessinger modified the milestones: next, v30.3.0 Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes Performance Component - Examples Affects the Examples module Component - Fatras Affects the Fatras module Event Data Model Infrastructure Changes to build tools, continous integration, ...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants