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

Reintroduce Dexter rhoV with grtrans mitigation #88

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bprather
Copy link
Collaborator

@bprather bprather commented May 5, 2023

Rotativity solutions for transport should correctly recover the low-temperature (non-relativistic) solution for Faraday rotation. However, there are various ways of transitioning from a fit designed to be broadly applicable down to the low-T limit.

This exists because for the polarized comparison, I wanted to directly compare low-temperature rotativity behavior from various codes. grtrans, for example, generally uses the fits outlined in the grtrans paper -- however, it has a somewhat interesting 3-part smooth transition to the non-relativistic limit, which ipole did not originally capture when using these functions.

Since it now behaves well (and quite similarly overall to the default Shcherbakov fit), this PR re-introduces the Dexter rhoV fit as a runtime option (previously, it could not be enabled at all). Basically, I found it lying around in a testing branch and figured it might be a net benefit to have upstream as a reference or comparison option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant