Build: #2 was successful Child of CASASPECS-DBC28-2
Code commits
CASA6
-
Akeem Wells 534b6388295983749c0988ac078d43d875be001c
Merge branch 'master' into CAS-14448
-
Srikrishna Sekhar 40b0c6f9a39c6d51211df799e2b5d5d0f2519730 m
Pull request #885: CAS-14201
Merge in CASA/casa6 from CAS-14201 to master
* commit 'b71d591b336f451eafdd6b5d603f185da4313b2b':
CAS-14201: Fix bug with mask=[]
CAS-14201 : Undo previous commit, with fix
CAS-14201 : Removed spurious immod call
CAS-14201 : Removed debug print statements
CAS-14201 : Intermediate commit, many cout statements
CAS-14201 : Fix peakResNM and peakRes swap
CAS-14201 - Multifield summaryminor no longer crashing
CAS-14201: Added pipes for minorcyclesummary in returndict
CAS-14201 - Partial commit for summaryminor returndict
CAS-14201 : niter > 0 peakres respects zero mask
CAS-14201 : Add check for mask validity
CAS-14201 : Bugfix for maskname
CAS-14201 : Roll back mask creation
CAS-14201 : Modify tclean(niter=0) to respect mask -
Srikrishna Sekhar b71d591b336f451eafdd6b5d603f185da4313b2b m
CAS-14201: Fix bug with mask=[]
If mask is passed in as an empty list, in the case of niter=0 the return
dictionary construction fell over. os.path.exists() wants an empty
string, but if an empty list was passed in it threw a KeyError.
This is now fixed by putting in the appropriate guard rails before it
gets to os.path.exists. The length of the input mask is tested, so
regardless of empty string or list it will catch it there.
This fixes the failing test case in simanalyze. -
Ihor Ihnatiev 13f6e3f0f3a27662d9da3568588317ae9c844e6e m
Pull request #884: CAS-14439
Merge in CASA/casa6 from CAS-14439 to master
* commit 'f11c3ddeec7842927b187f6a4c9dcee141eab55b':
RChanges according to the NAOJ code review Refs #2721 and Refs #2726 - emove dcolor codes from sdimaging deprecation warning due to log file readability issues
Changes according to the NAOJ code review Refs #2715 - changed line length
Removed test messages from warning
Deprecation warning for sdimaging task - added color highlight
Deprecation warning for sdimaging task -
Srikrishna Sekhar a50650a478503ef57ed15e9f517afd76579dbcdd m
Merge remote-tracking branch 'origin/master' into CAS-14201
-
Ihor Ihnatiev f11c3ddeec7842927b187f6a4c9dcee141eab55b m
Merge branch 'master' into CAS-14439
-
Akeem Wells 5c7ff663419900e4d4db774f37f36ad417f2abc6 m
Pull request #883: Update to use util find_spec instead of find_loader
Merge in CASA/casa6 from CAS-14470 to master
* commit 'acf17965893895c92835c7cabddf517ca0821f05':
Update to use util find_spec instead of find_loader -
Akeem Wells acf17965893895c92835c7cabddf517ca0821f05 m
Update to use util find_spec instead of find_loader
-
Ihor Ihnatiev c2c652557c4bbcde75987bd562dbd05cf6152cdc m
RChanges according to the NAOJ code review Refs #2721 and Refs #2726 - emove dcolor codes from sdimaging deprecation warning due to log file readability issues
-
Ihor Ihnatiev 71373facc12f93d3a065919b46d87119e8b6fa29 m
Changes according to the NAOJ code review Refs #2715 - changed line length
-
Ihor Ihnatiev 39c3e314e93844a91e9f01ab93aad706a4381e05 m
Removed test messages from warning
-
Ihor Ihnatiev 730c2050fe5d2004af3c3eb4592ca3acdccfae58 m
Deprecation warning for sdimaging task - added color highlight
-
Ihor Ihnatiev 138fe5d16d1e52fa4e5f393cc391d9c675f09502 m
Deprecation warning for sdimaging task
-
Srikrishna Sekhar 1c6da784040f6df79f260043f9a670e18fa3eaa6 m
CAS-14201 : Undo previous commit, with fix
Undoing the previous commit, and added the concomitant change in
SIMinorCycleController.cc. The changes to SIMinorCycleController were
only local, and hence my local tests were passing, but since they were
never pushed Bamboo tests were failing.
This is an effort to check if this fixes the Bamboo tests. -
Srikrishna Sekhar f04bca088eb4b83b112259b14bb9694426a2ea81 m
CAS-14201 : Removed spurious immod call
This was causing test failures in bamboo, and the original function call
results in the same intended behaviour. -
Srikrishna Sekhar 00b58867420d9f2ce0131963d371852072f5e6f6 m
CAS-14201 : Removed debug print statements
The previous commit appears to have solved all the issues. The unit
tests also pass (except for those marked to skip).
This commit removes the extraneous debug statements. -
Srikrishna Sekhar 1aa4f59e93ad2c19d0d874606c89d845fa037526 m
CAS-14201 : Intermediate commit, many cout statements
The issue with the broken tclean default behaviour has mostly gone away.
This commit is to preserve state as it is right now, a full explanation
will be in the next commit once I have had a chance to confirm by
running the test suite. -
Srikrishna Sekhar 8cb39e9e77d93dced5c9a8fbe69018d1ed3a85f4 m
Merge remote-tracking branch 'origin/master' into CAS-14201
-
Srikrishna Sekhar 78d904be5efa2759c781137ec4f006657126a261 m
Merge remote-tracking branch 'origin/master' into CAS-14201
-
Srikrishna Sekhar 0bbe6a8ca69dc39ec16c0eadf23bbcab64304cdf m
CAS-14201 : Fix peakResNM and peakRes swap
peakResNM and peakRes were swapped in the fullSummary=True return
dictionary. -
Srikrishna Sekhar ba5c7686483f12c3bbd33932494e07fc8041574f m
Merge remote-tracking branch 'origin/master' into CAS-14201
-
Srikrishna Sekhar 8850af82778b767f09332fbba3fbb9256b11dfa4 m
CAS-14201 - Multifield summaryminor no longer crashing
Specifying an outlierfield no longer crashes the return dict generation.
However for some reason tclean(niter=0) does the right thing with
multifield, i.e., reporting the results from each field separately,
while deconvolve(niter=0) and deconvolve(niter>0) only seem to report
the first field specified.- casatasks/src/private/imagerhelpers/imager_base.py (version 8850af82778b767f09332fbba3fbb9256b11dfa4) (diffs)
- casatasks/src/private/imagerhelpers/imager_deconvolver.py (version 8850af82778b767f09332fbba3fbb9256b11dfa4) (diffs)
- casatasks/src/private/imagerhelpers/imager_parallel_deconvolver.py (version 8850af82778b767f09332fbba3fbb9256b11dfa4) (diffs)
- casatools/src/code/synthesis/ImagerObjects/grpcInteractiveClean.cc (version 8850af82778b767f09332fbba3fbb9256b11dfa4) (diffs)
- casatools/src/code/synthesis/ImagerObjects/grpcInteractiveClean.h (version 8850af82778b767f09332fbba3fbb9256b11dfa4) (diffs)
- casatools/src/tools/iterbotsink/iterbotsink_cmpt.cc (version 8850af82778b767f09332fbba3fbb9256b11dfa4) (diffs)
- casatools/xml/iterbotsink.xml (version 8850af82778b767f09332fbba3fbb9256b11dfa4) (diffs)
-
Srikrishna Sekhar e573964754323d428da031f9436bb53d6a8bfb3d m
CAS-14201: Added pipes for minorcyclesummary in returndict
When niter > 0 and all-zero mask is passed in, the return dict now contains a
fully populated minorcyclesummary. At the moment this assumes a single field
(i.e., no outlier fields) which is hardcoded to 0. This will be fixed in a
subsequent commit.- casatools/src/code/synthesis/ImagerObjects/SIMinorCycleController.cc (version e573964754323d428da031f9436bb53d6a8bfb3d) (diffs)
- casatools/src/code/synthesis/ImagerObjects/SynthesisDeconvolver.cc (version e573964754323d428da031f9436bb53d6a8bfb3d) (diffs)
- casatools/src/code/synthesis/ImagerObjects/grpcInteractiveClean.cc (version e573964754323d428da031f9436bb53d6a8bfb3d) (diffs)
-
Srikrishna Sekhar 3c1a4cc979fe1ad217a58f801edcc38a5591362d m
CAS-14201 - Partial commit for summaryminor returndict
This commit partially implements the code needed to allow niter > 0 to
put in the summaryminor key into the return dictionary under the
condition of a zero-filled mask. -
Srikrishna Sekhar ae0e44455099aaa8935cf06ffa0a08fb941904df m
CAS-14201 : niter > 0 peakres respects zero mask
The `peakres` key within `summaryminor` now respects the user input mask
regardless of whether it is all zeros. If the user provides a mask where
a single plane (or all planes) are all mask=0, then correspondingly
peakres=0. This brings the niter > 0 in line with niter = 0 in this
regard. -
Srikrishna Sekhar bf0c64a33fc74e32bcad5a1cb4b5cc1b9fe1e29f m
Merge remote-tracking branch 'origin/master' into CAS-14201
-
Srikrishna Sekhar ed5303de2e37a5d1bb8f9b33069bda5fb46eb97e m
CAS-14201 : Add check for mask validity
If a mask is provided to `tclean(niter=0)` it now checks for the
existence _and_ validity of a mask image on disk. This fixes a bug where
a string/textfile passed into the `mask` parameter of tclean would
trigger it to try and open the mask with `ia.open`.
Now if the mask image is not present on disk, it gracefully fails.
Note that at this point it is not desirable to change the behaviour of
tclean(niter=0) to create a mask if none is present on the disk. -
Srikrishna Sekhar 844ecc6962bab995ab963b83a7f681b9070f9727 m
CAS-14201 : Bugfix for maskname
The previous commit assumed that the mask name is alwyas
`imagename.mask`. This fixes it to also allow passing in a mask via the
mask='' parameter inside tclean/deconvolve.
Verified that when a non-zero mask is provided, statistics are
appropriately calculated inside the mask. -
Srikrishna Sekhar 8b86b2eea2958da185f8e74eb8045eadcf5cdf75 m
CAS-14201 : Roll back mask creation
This rolls back the mask creation performed in the previous commit. As
of this commit - `tclean(niter=0)` will not create a mask, i.e., it
maintains existing behavior.
The statistics calculated for niter=0 will still respect an input mask
if provided - i.e., the statistics reported in the return dictionary
will respect the input mask. If no mask is provided, the stats are
calculated over the entire image. -
Srikrishna Sekhar c6d76ad807fff8056433644371c03ff3cded30c5 m
CAS-14201 : Modify tclean(niter=0) to respect mask
In the niter=0 case, task_tclean.py has been modified to now respect the
input mask if present. If there is no input mask present, a default mask
is generated (similar to the niter > 0 case) -- this is a
change from previous behaviour where tclean did not create a mask for
niter=0. This change now brings it in line with the behaviour of
task_deconvolve, that similarly creates a mask for niter=0
Currently the behaviour for a newly generated mask with niter=0 follows
that of niter>0 - i.e., if there is no mask present, and the "default"
is generated, for the purposes of computing statistics it is assumed
that the mask is filled with ones rather than zeros.