Build: #512 was successful Scheduled with changes by Srikrishna Sekhar
Code commits
Casa6
-
Srikrishna Sekhar 9eede3a21f1d4e6e61185ac7568ebc0a3563656f
Pull request #1056: CAS-13898
Merge in CASA/casa6 from CAS-13898 to master
* commit '81f8d50803a4f8d4ceb2d5dfecd8e67f25d11658':
CAS-13898 : Fix memory allocation per-plane
CAS-13898 : Re-use getMemoryAvailable from appRC
CAS-13898 : remove unnecessary temp image allocs -
Srikrishna Sekhar 81f8d50803a4f8d4ceb2d5dfecd8e67f25d11658 m
Merge remote-tracking branch 'origin/master' into CAS-13898
-
Srikrishna Sekhar ff635a9a893262d4b6ccfedd1e71fd9b3eb69a79 m
CAS-13898 : Fix memory allocation per-plane
The previous fix to pass in the available memory into auto-multithresh
did not work as expected, because the per-plane memory budget never
exceeds the total.
This fix divides the total memory budget by the estimated number of
planes used in the algorithm, and passes that in to the Lattice images.
This correctly seems to restrict the memory usage.
From initial benchmarking, memory limits are now respected as expected. -
Srikrishna Sekhar b21db203ff9a81fab1705c7a18cd11f9c9f9008a m
CAS-13898 : Re-use getMemoryAvailable from appRC
So now automultithresh should pick up the right memory limits
internally. -
Srikrishna Sekhar 67f8561f0cc6a4bbfda4c2a8172205b76ff29fcc m
CAS-13898 : remove unnecessary temp image allocs
Removed two temporary images that have memory allocated but are never
used. This seems to have brought down total memory usage by 20-30% for
auto-multithresh.