Build: #3 was successful Code changes detected

Stages & jobs

  1. Default Stage

  2. Test

Build result summary

Details

Completed
Queue duration
< 1 second
Duration
55 minutes
Labels
None
Revision
41c7dc77b583148c0117cf38598bd91a2884a594
Total tests
696
Successful since
#2 ()

Tests

Code commits

Author Commit Message Commit date
Rui Xue <rx.astro@gmail.com> Rui Xue <rx.astro@gmail.com> 41c7dc77b583148c0117cf38598bd91a2884a594 m CAS-14759: skip redundant normalizeprimarybeam() for cube+standard
For the standard gridder with specmode=cube, each worker in
CubeMajorCycleAlgorithm::task() already normalizes the primary beam
per-subcube during PSF making and writes the result back to the full
image. The subsequent master-side normalizeprimarybeam() loop over all
(pol x chan) planes was therefore redundant and, for a 7500-channel
8000x8000 cube, accounted for ~31 minutes of wall time after
"Time for making PSF and PB".

Changes:
- Extract the cube+standard guard into isCubeStandard bool for clarity.
- Skip the normalizeprimarybeam() loop when isCubeStandard is True.
- makePBCore() is unchanged and still skipped for cube+standard as before.
- Add a timing log entry "***Time for normalizeprimarybeam: <s> (skipped=<bool>)"
  at INFO3 so the optimisation is observable in tclean logs.

This path is safe to skip only for standard+cube because mosaicft and
awproject gridders derive PB from sqrt(weight)/pbmax and require the
fully-assembled weight image, which is only available on the master
after all subcubes have been written back.

Jira issues

IssueDescriptionStatus
Unknown Issue TypeCAS-14759Could not obtain issue details from Jira

Shared artifacts

Artifact File size
ManyLinux228 Casatestutils 144 KB
ManyLinux228 Python 3.12 Tool wheel 75 MB