-
Notifications
You must be signed in to change notification settings - Fork 5
Multipoles analysis #157
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
Open
p-slash
wants to merge
46
commits into
master
Choose a base branch
from
r-mu-binning-merge-master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+1,250
−736
Open
Multipoles analysis #157
Changes from all commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
9ec5bcd
initial implementation of new coordinates class
p-slash 629a721
comment out andrei recommended lines
p-slash 3e6472f
add few more attributes to prevent crashing
p-slash ccf0cf3
legendre compression attempt
p-slash 4127e2f
fix use multipoles and rmu bin check
p-slash 6389c67
convert model to multipoles by saving a matrix
p-slash f72f8fe
fixes to previous commit
p-slash cf02316
include data mask into multipole matrix. distortion matrix carries mu…
p-slash eb67a84
use csr_array instead of csr_matrix. multipole matrix is now a csr_ma…
p-slash 5a2fd61
change of attribute names for multipoles
p-slash b038377
distortion model has its own coordinates. need to recalculate multipo…
p-slash e070ecd
attempt at adding the bin smoothing effect
p-slash 87a5934
add mu bin smoothing
p-slash 07b92f0
turn off mu bin smoothing.
p-slash d2c97da
Option to apply Hartlap correction to cov. Requires NSAMPLES in the h…
p-slash 2d0c12c
smoothing legendre polynomials in the model to account for bin averag…
p-slash 3003686
fix: include the new option in init_from_Pk
p-slash e38f77d
try interpolating the metals matrix
p-slash 36a1c35
fixed and finalized r, mu binning of metals
p-slash ec92b87
upsample option for metals model when in rmu binning scheme
p-slash ebe8376
metal coordinates only support RtRpCoordinates for now
p-slash 7959114
fix regular mu grid when mu_min < 0
p-slash 54a5e56
global covariance support for xcf and hartlap correction
p-slash 346ac6f
calculate the percival correction for the global cov with Hartlap cor…
p-slash 158b78a
fix: do not fail for an empty global_cov_file string
p-slash 0d6e464
save ell and r for multipoles
p-slash a89a855
add percial correction to the output files
p-slash 43c9949
option to use "weighted" legendre multipoles.
p-slash b8531d5
fix epsilon values and limits
p-slash cc18151
use np nan for pad value
p-slash ad6d922
sum number of pairs in mu axis and enable saving it. Do not save ell …
p-slash e75a6d8
modify Minimizer and Output class to also save p_value
p-slash 813274a
additional keys in MODEL header to note datasize
p-slash 7477754
fix padding in output: Nan cannot be represented in integer arrays
p-slash 3ae8c14
introduce aiso aAP parameterization used by DESI galaxy clustering
p-slash 5ab7301
Turn off mu smoothing as I think it is included in Legendre decomposi…
p-slash 9e150bc
additional variables in FitResults's CorrelationOutput to for plottin…
p-slash fc9d8e3
rmu_metal_grid_factor is 1 by default
p-slash 874d32f
matrix to average data points for reference
p-slash 0a325ce
analysis changes regarding masked_fiducial. unsure about the purpose …
p-slash e65a2f5
fix merge conflict (minor)
p-slash c3a954b
Merge branch 'master' into r-mu-binning
p-slash 6a30431
fix attempt at output.py
p-slash 057b6c2
Update vega/scale_parameters.py. Typo in the docstring.
p-slash ad18f49
Update vega/output.py
p-slash c86d244
Update vega/output.py
p-slash 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
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.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
masked_fiducialis built usingfull_model_mask, but the global covariance Cholesky and noise vector are built usingfull_data_mask. Ifmodel_maskanddata_maskdiffer (e.g., due to distortion/model coordinate differences), this leads to shape mismatches when adding the noise term. Usefull_data_maskconsistently for the masked mean, or explicitly map the fiducial model from model-space to data-space before applying the global covariance.