HERA-Team / hera_pspec

HERA power spectrum estimation code and data formats
http://hera-pspec.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
5 stars 3 forks source link

cov_model and known_cov now parsed #332

Closed adeliegorce closed 3 years ago

adeliegorce commented 3 years ago

Resolves #327

Tested the code with the identity matrix as data covariance.

adeliegorce commented 3 years ago

Running the tests highlighted issues in parsing the covariance information (even for already existing options such as 'dsets' and 'autos'): if input_data_weighting is set to 'iC', for any input cov_model, it is always the empirical covariance that will be computed, therefore the R matrix does not change with cov_model. In particular, it needs to depend on the time_index and so the format of Rkeys needs to be changed.