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

Make sure injected L3 sources include Poisson noise #125

Open
schlafly opened this issue May 28, 2024 · 0 comments
Open

Make sure injected L3 sources include Poisson noise #125

schlafly opened this issue May 28, 2024 · 0 comments
Assignees

Comments

@schlafly
Copy link
Collaborator

I had forgotten that if non-chromatic sources are used, add_objects_to_image doesn't include Poisson noise and simulates a noise-free image.
https://github.com/spacetelescope/romanisim/blob/main/romanisim/image.py#L215-L221
This leads the L3 source injection approach to not actually inject Poisson noise. We should probably add an option to add_objects_to_image to include Poisson noise.

@PaulHuwe PaulHuwe self-assigned this Jun 26, 2024
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

2 participants