Skip to content

fix(possum_prefect): rebuild to pick up healpy-based possum2caom2 (fixes 0923 vs 0922 tile names)#1

Merged
Sebokolodi merged 1 commit into
CIRADA-Tools:mainfrom
Sebokolodi:fix/possum2caom2-healpy-rebuild
May 26, 2026
Merged

fix(possum_prefect): rebuild to pick up healpy-based possum2caom2 (fixes 0923 vs 0922 tile names)#1
Sebokolodi merged 1 commit into
CIRADA-Tools:mainfrom
Sebokolodi:fix/possum2caom2-healpy-rebuild

Conversation

@Sebokolodi

Copy link
Copy Markdown
Collaborator

What

Adds a dated comment above the possum2caom2 install in possum_prefect/Dockerfile to trigger a rebuild that picks up the current possum2caom2@main.

Why

The image's baked possum2caom2 dated from Aug 2025 (v1.15.2), predating the switch from astropy_healpix to healpy. With the old library, HEALPix boundary tiles get the wrong coordinate string in their filenames — e.g. tile 6514 is named …0923-0335… instead of …0922-0335….

Root cause: astropy_healpix.healpix_to_lonlat(6514) returns RA = 140.625 exactly → the minute rounds up to 23; healpy.pix2ang returns 140.6249999… (1 ULP lower) → rounds to 22. possum2caom2@main now uses healpy, so a fresh rebuild produces the correct 0922.

Notes

  • @main is intentionally kept (no version pin); CD builds with no-cache: true, so the rebuild re-pulls possum2caom2@main fresh.
  • Patch-level change (fix:).

After merge

  • Builder action rebuilds/pushes the new image to Harbor.
  • Repoint the Prefect local-pool worker at the new image tag and restart it.
  • Re-rename the legacy …0923… tiles already in the archive (the pipeline's _rename skips already-renamed files).

…xes 0923 vs 0922 tile names)

The baked possum2caom2 dated from Aug 2025 (pre-healpy switch), so the image still ran astropy_healpix and named boundary tiles like tile 6514 as 0923 instead of 0922. possum2caom2@main now uses healpy; CD builds with no-cache so this rebuild re-pulls it fresh.
@Sebokolodi
Sebokolodi merged commit ce53403 into CIRADA-Tools:main May 26, 2026
1 of 2 checks passed
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.

1 participant