Skip to content

work on tryUpdates structure#124

Open
dstndstn wants to merge 13 commits into
craig_factored_mergefrom
try-updates
Open

work on tryUpdates structure#124
dstndstn wants to merge 13 commits into
craig_factored_mergefrom
try-updates

Conversation

@dstndstn

@dstndstn dstndstn commented Jul 11, 2025

Copy link
Copy Markdown
Owner
  • for tryUpdates, factored out getting "alpha" step sizes to try
  • rename BatchImageDerivs to BatchGalaxyProfiles
  • factor out galaxy-rendering from derivative-computing
  • factor out common creation of BatchGalaxyProfile and BatchImageParams
  • use resulting functions in tryUpdate, get up to rendering galaxy profiles (but not computing log-probs)

dstndstn and others added 13 commits July 10, 2025 07:27
…ethod to choose the step sizes & parameter updates
- rename BatchImageDerivs to BatchGalaxyProfiles, removing some terms only required for derivatives
- factor out code to produce BatchGalaxyProfiles and BatchImageParams in factored_optimizer.py
- factor out code that renders galaxy profiles
- sketch out the rest of tryUpdates
by using getImage(use_gpu=True) and getInvError(use_gpu=True)
rounding error propagation from tractor/miscutils.py in lanczos filter
logic.

Use constrained_optimizer.py tryUpdates for now even though tryUpdates
is now implemented in factored_optimizer.py since there are slight
differences.

Use GPU copies when avaialble of arrays and cache them.

Float32 and complex64 and del arrays when done to make sure no
OutOfMemoryExceptions.

In batch_mixture_profiles, chunk data for 5D array F along dimension
used to sum to Fsum for large arrays.

Use gpu_lsqr_optimizer.py for background fitting.
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