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

Need for new scattering models #10

Open
KAHR-Alpha opened this issue Aug 12, 2023 · 4 comments · Fixed by #12
Open

Need for new scattering models #10

KAHR-Alpha opened this issue Aug 12, 2023 · 4 comments · Fixed by #12
Assignees
Labels
enhancement New feature or request

Comments

@KAHR-Alpha
Copy link
Owner

The implementation of the Lambertian, Guassian and ABg scattering models has been requested. ( see https://support.zemax.com/hc/en-us/articles/1500005575662-What-scattering-models-are-available-in-OpticStudio- )

@KAHR-Alpha KAHR-Alpha added the enhancement New feature or request label Aug 12, 2023
@KAHR-Alpha
Copy link
Owner Author

@iromero91 it seems everywhere I look the models are reflective only. Did you use them in transmission?

@KAHR-Alpha KAHR-Alpha self-assigned this Aug 12, 2023
@iromero91
Copy link
Collaborator

The main purpose i need them for is total internal reflection, but transmission too. The models in that zemax doc just affect the ray as it would come out of the surface, either refracted or reflected, shouldn't matter.

@KAHR-Alpha
Copy link
Owner Author

Yeah eventually that's what I saw on the page about fitting the ABg coefficients. One thing I still don't get is how reflection and transmission are coupled through the model.

Total internal reflection is a good example. One would expect some light to be scattered through the surface, but not according to that algorithm.

@KAHR-Alpha KAHR-Alpha linked a pull request Oct 9, 2023 that will close this issue
@KAHR-Alpha KAHR-Alpha reopened this Oct 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants