Skip to content

Comments

Up the ramp sampling#74

Open
arunkannawadi wants to merge 5 commits intomainfrom
photon_sampling
Open

Up the ramp sampling#74
arunkannawadi wants to merge 5 commits intomainfrom
photon_sampling

Conversation

@arunkannawadi
Copy link
Member

@arunkannawadi arunkannawadi commented Feb 16, 2026

@Higgins00
Copy link
Contributor

Higgins00 commented Feb 16, 2026

Required github branch for operation is photon_stamps in https://github.com/DukeCosmology/GalSim.git

@Higgins00 Higgins00 marked this pull request as ready for review February 16, 2026 20:52
@Higgins00 Higgins00 force-pushed the photon_sampling branch 6 times, most recently from 061acd7 to 57bdf6c Compare February 17, 2026 18:15
…ile locations that are dependent on system"

This reverts commit a07b01b.
…arguments but instead have the draw function return the photon array

Added makefromimage function to fft generated galaxies for their photonarray. General concept for a way to do the sampling, but all of this will likely need to be wrapped in its own method or Buildfromstamps will need to be modified to accept photonarray as an output.

Removing file I previously made to test ways to do the photonarray combination

changes to sca.py to just generate an image from photon arrays

Stamp now is just for the photon shooting. The sca.py has been changed to conver a dt photon array into an image.

Generate an image of 1/20th an exposure using full image photon arrays. Corrected the flux scaling issue for the photons resulting in a brighter image than expected, changed photon array concatenation as well.

removed egg-info and __pycache__

added a input register and value register for the resultants strategy.

Implemented the resultant image builder. resultant image headers, BFE, optimizations still need to be done

Some comments

Separate classes so it can be  merged with main.

fixing formating and preparing for merge into main
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.

2 participants