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

What if scattering or absorption factors are greater than one? #2

Closed
2 tasks done
fiedl opened this issue Jan 24, 2018 · 2 comments
Closed
2 tasks done

What if scattering or absorption factors are greater than one? #2

fiedl opened this issue Jan 24, 2018 · 2 comments

Comments

@fiedl
Copy link
Owner

fiedl commented Jan 24, 2018

Currently, the --scattering-factor and the --absorption-factor are assumed to be between zero and one. If zero, the photons interact instantly when entering the hole ice. If one, the photons interact the same way as in the surrounding ice.

But actually, the factor can be greater than one, meaning that the interaction probability within the hole ice would be less than in the surrounding ice.

@fiedl fiedl mentioned this issue Feb 3, 2018
9 tasks
@fiedl
Copy link
Owner Author

fiedl commented Feb 4, 2018

Some thoughts on the geometric cases: When the interaction factors are allowed to be greater than 1 then a switch for the special cases "2b" and "3b" is needed.

img_4386

@fiedl
Copy link
Owner Author

fiedl commented Feb 4, 2018

Currently, I'm using nan to indicate not only that an intersection point does not exist, but also that an intersection point is not between the interaction points A and B of the photon.

In "case 3b", I need access to YC, which is not available in this case, because it requires termination_point_ratio, which is set to nan because the photon ends inside the hole ice.

Probably it would be better to not cut off the ratios if they are <0 or >1. This would make all necessary values available to "case 3b" as well.

For the calculation of the number of medium changes, one would need to check whether the entry_point_ratio and the termination_point_ratio are between 0 and 1 rather than checking for nan.

fiedl added a commit to fiedl/clsim that referenced this issue Feb 23, 2018
This increases precision and performance, but requires extensive
refactoring. This aims to fix
fiedl/hole-ice-study#28.

On the plus side, this helps removing the un-needed old code mentioned
in
fiedl/hole-ice-study#2 (comment).

Using native vector operations also requires to re-check the
xy-projection. Fortunately, there are still tests in place for that.
See: fiedl/hole-ice-study#25.
@fiedl fiedl closed this as completed Feb 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant