-
Notifications
You must be signed in to change notification settings - Fork 0
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
Implement G4Trap in Opticks #5
Comments
G4Trd does not need to be implemented since it can be translated analytically into a G4Trap. |
Can I assign it to you @plexoos ? |
Yes, I'll see what I can do |
We have both G4Trap and G4Trd (aka |
That is correct, I provided a way to automatically convert all G4Trds into G4Traps. I can not find it right now but will add a comment here when I do. Hence only G4Traps need to implemented in this task into Opticks. |
Looks like we can associate up to 6 double parameters with each node https://github.com/BNLNPPS/esi-opticks/blob/esi/sysrap/s_pa.h#L50. But |
Yes, I feel like we have to change this sourcecode and add additional variables which will lead to need to change quite a bit of code... Simon suggested the following steps to do this:
|
We are in luck. All of the trap volumes in pfrich_min have zero angles:
|
I saw your commit @plexoos, very interesting! Did you test it? Does it work or does G4Trap need to be converted into Polyhedron and have a function to calculate it's bounding box for GPU use? By the way do you know what sn and snd stand for? |
AFAICT, sn/snd are the types that hold information about each node in the geometry tree. I implemented the right angular wedge from the trapezoid definitions, and it seems to allow processing a simplified geometry with a union that includes a trapezoid. More tests are needed. |
The intersection is implemented for Polyhedron for Optix7! |
No description provided.
The text was updated successfully, but these errors were encountered: