-
Notifications
You must be signed in to change notification settings - Fork 11
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
New method for computing optical depths #93
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ork too. Superlinear power laws for temperature seem to result in a minor offset compared to the default feautrier solver, but in that case I do not know which is more accurate.
…is obtained. Furthermore tried to write own specialized version of erf, but the compiler wins.
…tau methods should still be implemented properly TODO.
This was referenced Jul 15, 2022
…ute the optical depth.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Implemented a new way of computing the optical depth by integrating the opacity over an individual line. This changes the way how rays need to be handled (no longer do we need to add ghost points in between if the doppler shift is too large). For models with moderate shifts, this can be about 10% slower, however, for larger doppler shifts, this results in a massive performance increase (no longer does the computation time scale with the total shift on the ray). On the plus side, the optical depth should be computed more accurately. For playing around with speed vs accuracy, one can always change the parameter max_width_fraction (higher->less accurate, but faster).
Practically, most solvers just have some minor code changed to use the new method. However, the short-characteristics method has been (lazily) rewritten to use the source function and optical depth instead of emissivity and opacity. The inspection function for the feautrier solver has also been slightly changed and now no longer returns the emissivity and opacity (instead return the source function).