Skip to content

gh-566: add example on galaxy redshift distributions #567

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 10 commits into from
Apr 15, 2025
Merged

gh-566: add example on galaxy redshift distributions #567

merged 10 commits into from
Apr 15, 2025

Conversation

ntessore
Copy link
Collaborator

@ntessore ntessore commented Mar 6, 2025

Description

Add an example that describes the caveats for sampling galaxy redshift distributions from a set of discrete matter or lensing shells.

Closes: #566

Changelog entry

Added: example on effective galaxy redshift distributions

@ntessore
Copy link
Collaborator Author

ntessore commented Mar 6, 2025

Copy link
Member

@Saransh-cpp Saransh-cpp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me, thanks, @ntessore!

Copy link
Member

@paddyroddy paddyroddy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few comments

Copy link
Contributor

@arthurmloureiro arthurmloureiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!

It did take me reading the full example to understand what it was about. Maybe we could make it more clear that this example shows how the definition of the matter shells affects the final redshift distribution? I am saying this but I do not have a clear suggestion on how to make it clearer haha.

@ntessore
Copy link
Collaborator Author

It did take me reading the full example to understand what it was about. Maybe we could make it more clear that this example shows how the definition of the matter shells affects the final redshift distribution? I am saying this but I do not have a clear suggestion on how to make it clearer haha.

@arthurmloureiro Your sentence makes it clear, so I added it to the top of the notebook

Copy link
Contributor

@arthurmloureiro arthurmloureiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a green light from my side! Nice and clear example!

@NiallJeffrey
Copy link

There is a phrase in the text in bold:
“The redshifts assigned to each individual galaxies are completely irrelevant here. The clustering and lensing signals are fully determined by the shells.”
I’d suggest
(I) changing this to “Within a particular shell, the individual galaxy redshifts are effectively dummy variables. The clustering and lensing signals are fully determined by the definition of the shell (and these internal redshifts have no impact).”
(II) Placing this block of text as the second paragraph right at the top. My original confusion was about some slightly vague wording, but this is completely removed if this text is at the start providing clarification before the technical stuff -- with this present wording (if you agree).
The demo itself is very clear 👍

@ntessore
Copy link
Collaborator Author

Thanks @NiallJeffrey, that makes the message much clearer. Could you review again?

Copy link

@NiallJeffrey NiallJeffrey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great 👍

@ntessore ntessore merged commit 486c81b into main Apr 15, 2025
17 checks passed
@ntessore ntessore deleted the nt/566 branch April 15, 2025 11:53
@ntessore
Copy link
Collaborator Author

Thanks all!

@ntessore
Copy link
Collaborator Author

@all-contributors please add @arthurmloureiro for review

Copy link
Contributor

@ntessore

We had trouble processing your request. Please try again later.

@ntessore
Copy link
Collaborator Author

@all-contributors please add @NiallJeffrey for review

Copy link
Contributor

@ntessore

We had trouble processing your request. Please try again later.

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

Successfully merging this pull request may close these issues.

Add an example on effective redshift distributions
5 participants