Skip to content

Ngmix v2.0#740

Open
martinkilbinger wants to merge 57 commits into
CosmoStat:developfrom
martinkilbinger:ngmix_v2.0
Open

Ngmix v2.0#740
martinkilbinger wants to merge 57 commits into
CosmoStat:developfrom
martinkilbinger:ngmix_v2.0

Conversation

@martinkilbinger
Copy link
Copy Markdown
Contributor

@martinkilbinger martinkilbinger commented Jun 1, 2026 •
edited
Loading

Summary

  • Running ShapePipe for UNIONS v2.0 including shape measurement.
  • Upgrade ngmix to 2.4.0, and use @lbaumo's new ngmix classes and interface.
  • Deleted obsolete scripts and config files.
  • Validation tests of centroid bugs using different versions of the code

Reviewer Checklist

Reviewers should tick the following boxes before approving and merging the PR.

  • The PR targets the develop branch
  • The PR is assigned to the developer
  • The PR has appropriate labels
  • The PR is included in appropriate projects and/or milestones
  • The PR includes a clear description of the proposed changes
  • If the PR addresses an open issue the description includes "closes #"
  • The code and documentation style match the current standards
  • Documentation has been added/updated consistently with the code
  • All CI tests are passing
  • API docs have been built and checked at least once (if relevant)
  • All changed files have been checked and comments provided to the developer
  • All of the reviewer's comments have been satisfactorily addressed by the developer

Lucie Baumont and others added 30 commits January 9, 2023 14:07
ngmix update runs, with issues
f27d4c2
still debugging ngmix
b36b92b
Merge remote-tracking branch 'origin/develop' into ngmix_update
4ea9b42
change T to r50
56e60ba
added r50 as galaxy guess option
d86c88a
new version of ngmix produces output. not yet tested the quality
52e411e
@lbaumo
unfinished overhaul of ngmix module
9b7c8d7
@lbaumo
ngmix module overhaul
441797e
@lbaumo
Merge branch 'CosmoStat:develop' into ngmix_update
2b34488
@lbaumo
ngmix update psf stuff
396035c
@lbaumo
ngmix module random numbers
139ffae
@lbaumo
ngmix module new classes
ccf6e1a
@lbaumo
Merge branch 'ngmix_update' of https://github.com/lbaumo/shapepipe in…
e09dc48 
…to ngmix_update
@lbaumo
more ngmix updates
75ddcc8
@lbaumo
ngmix update
a98063a
Added ngmix test fitting script
886e0db
docker install: leave extensions
8fd590c
remove obsolete scripts
621bab1
keeping extension for shapepipe_run.py, updated workflow
06934c9
Fixed Dockerfile
883c4d6
@claude
fix: add bin/ wrappers to git and fix pyproject.toml license format
a662554 
- bin/ scripts were untracked, causing Docker build to fail
- Fix license field to use SPDX string format (MIT) to resolve
  SetuptoolsDeprecationWarning

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@claude
fix: use ENV key=value format in Dockerfile
ce6804d 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
fixed extensions of scripts
88f3fe3
@martinkilbinger
added missing Git_cat cfg file
25d9238
modified Mh runner acfg
8bcfe1f
going to job 512
a9e70c6
added ID to Mh output
a3eec86
added ID to Mh output
a813280
v2.0 running up to job 256
5856b9f
@martinkilbinger
fixing log headers; weightwatcher exe
adc2684
martinkilbinger and others added 27 commits May 6, 2026 13:13
@martinkilbinger
Merge remote-tracking branch 'upstream/develop' into ngmix_v2.0
f922f05
improved job checks
b0c6f31
multi-epoch vignet handling for v2.0
7fccb0e
@martinkilbinger
Merge branch 'ngmix_v2.0' of github.com:martinkilbinger/shapepipe-1 i…
546e628 
…nto ngmix_v2.0
@martinkilbinger
dell changes
8bd99d6
@martinkilbinger
Merge remote-tracking branch 'origin/ngmix_v2.0' into ngmix_v2.0
e500139
@martinkilbinger
Merge remote-tracking branch 'upstream/develop' into ngmix_v2.0
0686c55
@martinkilbinger
cleaned up cfg files; job 512 seems to be working using ; fixed some …
12254cd 
…bugs when exp list is empty
@martinkilbinger
running until ngmix call
bed3a50
@martinkilbinger
Merge remote-tracking branch 'remotes/origin/ngmix_update' into ngmix…
6839230 
…_v2.0
@martinkilbinger
Update to ngmix v2.4;
d01503d 
Fix check on zero-valued pixels (most did not pass), now all vignets with
at least one != 0 pixel pass.
Added Axel's centroid shift test scripy (py)
fd88094
Merge branch 'ngmix_v2.0' of github.com:martinkilbinger/shapepipe-1 i…
ceb4f91 
…nto ngmix_v2.0
@martinkilbinger
debugging ngmix and small nb of processed objects
7e00cfd
@martinkilbinger
Merge branch 'ngmix_v2.0' of github.com:martinkilbinger/shapepipe-1 i…
eba7ece 
…nto ngmix_v2.0
adapting centroid shift script to v2.0
b729eb8
Add centroid bias validation infrastructure from test_centroid_bug
9f4170d
@martinkilbinger
Merge remote-tracking branch 'origin/ngmix_v2.0' into ngmix_v2.0
7f3d172
testing centroid bug and fix with three different setting (bug, fix, …
bd12880 
…v2.0)
centroid bug fixed for v2.0
440f822
fixed pyproject for Docker deployment
0f52921
fixing versions in docker
3a70fdf
@martinkilbinger
Merge remote-tracking branch 'origin/ngmix_v2.0' into ngmix_v2.0
77054bb
@martinkilbinger
fixing bugs in ngmix
60962ff
@martinkilbinger
Merge remote-tracking branch 'origin/ngmix_v2.0' into ngmix_v2.0
30181ff
@martinkilbinger
fixed merge errors
d146cae
@martinkilbinger
debugged ngmix
67079de
@martinkilbinger martinkilbinger self-assigned this Jun 1, 2026
@martinkilbinger martinkilbinger added the enhancement New feature or request label Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cailmdaley cailmdaley Awaiting requested review from cailmdaley

Assignees

@martinkilbinger martinkilbinger

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@martinkilbinger @lbaumo