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

Enhance support for symmetries in potential calculation #215

Open
lmh91 opened this issue Sep 13, 2021 · 1 comment
Open

Enhance support for symmetries in potential calculation #215

lmh91 opened this issue Sep 13, 2021 · 1 comment
Labels
enhancement Improvement of existing features
Milestone

Comments

@lmh91
Copy link
Collaborator

lmh91 commented Sep 13, 2021

Right the user is able to define the world in the config file.
Symmetries can be used there. For example mirror symmetries.

The issue is that the symmetries might differ for different things to simulate.
For example, the example coax detector has in general a periodicity of 60° in φ.
In addition, within this period, it has a mirror symmetry.
Thus, the electric potential only needs to be calculated on the interval [0°, 30°] (with reflecting boundary conditions in φ).

However, this is not possible for the weighting potentials of individual segments. Those, in this case, have
only a mirror symmetry but not a periodicity. Thus, they are calculated over 360°.
It could be reduced to [x°, x°+180°].

For this we need to add support for defining those symmetries in the configuration files for each individual contact.

@lmh91 lmh91 added the enhancement Improvement of existing features label Sep 13, 2021
@lmh91 lmh91 added this to the v1.0.0 milestone Sep 13, 2021
@fhagemann
Copy link
Collaborator

fhagemann commented Sep 17, 2021

Concerning this: can we handle stuff like ConeMantle going from -30° to 30°? The angle in CylindricalPoint always goes from to 360°, so I guess a point at 350° will return false when checking if it is in [-30°, 30°].

@fhagemann fhagemann changed the title Enhance support for symmetries Enhance support for symmetries in potential calculation Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement of existing features
Projects
None yet
Development

No branches or pull requests

2 participants