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

Transfer function API and integration fixes #143

Merged
merged 2 commits into from
Aug 21, 2023
Merged

Transfer function API and integration fixes #143

merged 2 commits into from
Aug 21, 2023

Conversation

fjebaker
Copy link
Member

@fjebaker fjebaker commented Aug 21, 2023

Attempt to fix different problems with the transfer functions.
Screenshot 2023-08-21 at 11 22 05

The seam in this figure corresponds to the bin edge $g^\ast \in [0, h]$ and $g^\ast \in [1-h, 1]$. I tried a few different schemes to resolve this, but I think it's a combination of getting the timing wrong, and also the flux in the bin edges being too low with that Dauser approximation?

@fjebaker fjebaker merged commit 89d7288 into main Aug 21, 2023
1 check passed
@fjebaker fjebaker deleted the fergus/tf-fixes branch August 21, 2023 10:39
@phajy
Copy link
Contributor

phajy commented Aug 21, 2023

Very interesting. What value of $h$ are you using (and does it vary with $r_e$)? Can you quantify the glitch, e.g., by plotting the flux contribution / time as a function of $g^\star$? Might help see if the problem is mainly with flux rather than time.

It looks more prominent for $g^\star \simeq 0$ which is the locus of points with minimum redshift at each radius on the disc. I guess you could look at a disc image of $g$ and see how things look near the line $g^\star(r_e) = 0$ (i.e., $g_\text{min}(r_e)$ ), compared to $g^\star(r_e) \simeq h$.

Sorry, you've probably already tested a bunch of things. A slightly frustrating problem!

@fjebaker
Copy link
Member Author

fjebaker commented Aug 21, 2023

Hi @phajy! Thanks for the tips :)

In the above plot $h = 10^{-8}$, but the seam persists for any order of $h$.

The line profiles themselves seem to be unaffected (as in, summing along the time axis reconstructs the line profile correctly). It's definitely something to do with the binning and how the code is currently calculating which time bin to put the flux in, and less likely to do with the flux calculated in the edge.

A couple of observations

  • everything to the left of the seam is the upper branch, everything to the right is the lower branch of both the $t$ and $f$ transfer function loops
  • setting the edge contributions to zero removes everything between the brightest two ridges of the seam (here for a small span of radii)
    Screenshot 2023-08-21 at 18 56 54
    but the seam is still visible, further implying the problem is related to time bin calculations.

fjebaker added a commit that referenced this pull request Aug 22, 2023
Transfer function API and integration fixes
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.

None yet

2 participants