-
Notifications
You must be signed in to change notification settings - Fork 10
fix: better init of spergel HLR and tabulate xValue #258
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
Merged
Merged
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
bf7ba9b
fix: better init of spergel HLR
beckermr 7ebdd85
pytest 8
beckermr 33db36f
fix: get ci right for benchmarks
beckermr 973eaf2
perf: smaller interval
beckermr 90b5194
perf: use galsim to init on CPU if we can
beckermr 5809fe9
fix: make sure we have debug symbols
beckermr 9db82e7
perf: use interp for spergel in real-space
beckermr e58adb0
fix: remove debug statement
beckermr 646dca7
fix: handle zero correctly
beckermr ba209a6
fix: use slow function
beckermr 1f5e953
perf: try a mix of asymp, interp, and small-scale expansion
beckermr 65db956
fix: same small nu function
beckermr 65bb443
fix: match things smoothly
beckermr efd3f58
Apply suggestion from @beckermr
beckermr 6ac2821
perf: see how slow this can be
beckermr 895b698
Merge branch 'test-spergel' of https://github.com/GalSim-developers/J…
beckermr 340a5cf
perf: tight starting bounds
beckermr c6837ca
Merge branch 'main' into test-spergel
beckermr ef1603d
Merge branch 'main' into test-spergel
beckermr b89fbe5
test: add test showing it is the FFT size for rendering
beckermr 5c40786
fix: adjust test tolerance for mismatched FFT sizes
beckermr cc1ef81
fix: remove exact branch
beckermr 944ad91
fix: remove changes not needed here
beckermr 50f25b9
fix: add comments and cleanup code
beckermr 9b75875
fix: address code review
beckermr 74c16eb
test: add one test of the profile directly
beckermr 1d2117c
perf: use log-log interp
beckermr db604d6
fix: negative flux is ok
beckermr File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.