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

All geometry files need to have the updated target position #8

Open
cameronc137 opened this issue Dec 14, 2018 · 2 comments
Open

All geometry files need to have the updated target position #8

cameronc137 opened this issue Dec 14, 2018 · 2 comments

Comments

@cameronc137
Copy link
Collaborator

I copied the prex2.gdml file to add my new SAMs into, but it didn't have the updated target position, so my previous update simulation results are scaled up by the relative amount due to this target position issue (which probably isn't a problem as long as you use ratios, but the absolute number is wrong and should be updated).

<constant name="prex_target_center" value="-105.3"/>

@cameronc137
Copy link
Collaborator Author

Also, this target position should be update in this line 49, not in the physical volume placement itself.

@cipriangal
Copy link
Owner

No it shouldn't because this gdml file is a mess and has a lot of interconnected parts. All I want to change when I update the target position is the target .. not the coils of the septum or something like that. I do not have the time to track down all the tgt position dependencies. In in ideal world I agree this should be fixed.

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

No branches or pull requests

2 participants