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

Generalise the current Factory interface for any Nx2pt factories (including GC multi-tracers) #480

Open
arthurmloureiro opened this issue Jan 15, 2025 · 1 comment

Comments

@arthurmloureiro
Copy link

Hi @marcpaterno and @vitenti !

(Putting our discussion from yesterday into an issue as you suggested)

The current implementation for the TwoPoint.from_metadata accepts a wl_factory and a nc_factory with other 2pt probes such as CMB Lensing to be added as something like cmbl_factory due to internal checks in this function:

What we seemed to have converged on our hacking telecon was to modify this method:

def _from_metadata_single_base(cls, metadata, wl_factory, nc_factory):

Allowing us to call a list of different factories:

all_two_point_functions = tp.TwoPoint.from_metadata(
    metadata_seq=all_two_point_cells,
    factories=[ncf1, ncf2, wlf, cmblf],
)

where the combination of which probes to cross-correlate with are inside the all_two_point_cells metadata.

I also understand we finished the telecon with the understanding that currently adding this support would not fix the support for GC multi-tracers due to their measurements being the same Galaxies.COUNTS, but I think @vitenti had a potential solution to this.

@arthurmloureiro
Copy link
Author

Tagging also @VGozzi here as he as experience with ~hacking GC Multi-tracers in Firecrown :)

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

No branches or pull requests

1 participant