-
Notifications
You must be signed in to change notification settings - Fork 7
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
few fixes #376
base: master
Are you sure you want to change the base?
few fixes #376
Conversation
marina-ricci
commented
Nov 7, 2024
- we still need to figure the coordinate problem
- the CLMM version for installed is not updated apparently
- we need to clean the repo
- merge with Tomomi's working branch
- understand how to use true shear
- understand how to use full cosmoDC2 datasets
…Pipe into clmm-update-nov-2024
Hi @marina-ricci, I just added the code to convert the output in |
I've added you to the TX team, so you should have access now @eduardojsbarroso |
@marina-ricci However I have two points on the code I just pushed:
|
@marina-ricci I took the code I had on the notebook and created a Is there other parts of this pull request I can work on so we can merge? |
The failing test was just because we were testing TXTwoPointRLens in the redmagic pipeline file. I've removed it since it wasn't really needed there. |
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.
I'm pre-approving this so CL people can merge if they want to, since it only affects the extension module.