iat-cener / tonatiuh

A Monte Carlo ray tracer for the optical simulation of solar concentrating systems
http://iat-cener.github.io/tonatiuh/
GNU General Public License v3.0
54 stars 14 forks source link

Ray leakage #83

Open JJP660 opened 7 years ago

JJP660 commented 7 years ago

Hi

Please see the attached image. Is there any way to prevent rays 'leaking' through imported STL surfaces? ray_leak

Thanks

ilescener commented 7 years ago

I'll check it, something seems wrong, could you attach the CAD file or the Tonatiuh file with the geometry?

JJP660 commented 7 years ago

Hi

Please find the Tonatiuh file attached.

Regards


From: ilescener notifications@github.com Sent: Friday, 28 October 2016 5:37:23 PM To: iat-cener/tonatiuh Cc: Johannes Pottas; Author Subject: Re: [iat-cener/tonatiuh] Ray leakage (#83)

I'll check it, something seems wrong, could you attach the CAD file or the Tonatiuh file with the geometry?

You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/iat-cener/tonatiuh/issues/83#issuecomment-256847321, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATsePEwiVnk4lqjZwVqSWHiMAJyJa-jWks5q4ZgjgaJpZM4KjFKy.

ilescener commented 7 years ago

I can't see the file. Sometimes happens that the file doesnt' get attached if people reply the mail instead of comment inside the post at github. Could you send it to my email address? iles@cener.com

Regards

ilescener commented 7 years ago

Dear user,

The intersection between the CAD surface and the rays doesn't perform correctly because of the small size of the triangles that compound the surface (some tolerances don't fit to these small sizes), I can fix the code but unfortunately I don't think that a new version with this issue fixed will be available soon. Meanwhile, I recommend you to generate the surfaces using cm or mm as unit to make them longer for Tonatiuh and probably the issue will disappear automatically.

Thank you very much for reporting these bugs and sorry for the inconveniences.

Iñigo