AST-1567 Add integration test to WGridderPredict
The integration tests are based on the integration tests for IDGPredict ddecal/test/integration/tIDGPredict.py
.
The tests that do not involve DdeCal have been placed in steps/test/integration/tWGridderPredict.py
The tests do use DdeCal have been placed in ddecal/test/integration/tWGridderPredictDdeCal.py
The following features have been added
-
Padding to a multiple of four. Image sizes that are not a multiple of four were not supported. The restriction comes from the faceting routines in schaapcommon. WGridderPredict now pads these images to a multiple of four. There are no automated tests for this feature, but @nicolas-slusarenko has used this feature in his demonstration. (TJD: this was added in !1305 (merged))
-
Store directions for extra data buffer in DpInfo. DdeCal needs to know the direction associated with the model data, to correctly write the solutions to h5parm. DpInfo now contains a map of strings to directions that is filled by WGridderPredict. DpInfo simply exposes the entire map. The alternative is to add functions for adding, removing and looking up entries. This is how DpBuffer exposes the extra data buffers. However, these functions together also provide full access to the map, just like exposing the map, since inserting, removing and looking up is everything that can be done with a map. There are no unit tests for this functionality, but it is used in the integration test. (TJD: this was added in !1304 (merged))
Merge request reports
Activity
assigned to @tol
added 36 commits
-
b67d1970...7f309433 - 4 commits from branch
master
- 7f309433...1634fe93 - 22 earlier commits
- f498b5f7 - Fix typo in comment
- e29e8001 - Apply 1 suggestion(s) to 1 file(s)
- bbd7abf9 - Clean up includes and usings
- 12128b6b - Order includes
- 363eca80 - Remove unused code, add some constants and comments
- 7b7afc07 - Apply 2 suggestion(s) to 1 file(s)
- 858cd717 - Add integration test for WGridderPredict step
- a281fc1b - Add model image padding to aligment of 4
- c1b0a2d7 - Use padded size when reading facets
- b4d42946 - Add nthreads parset key
Toggle commit list-
b67d1970...7f309433 - 4 commits from branch
added 13 commits
-
48e0cda9...c62e00da - 6 commits from branch
master
- d1621655 - Add integration test for WGridderPredict step
- 92c87197 - Add model image padding to aligment of 4
- 024df96f - Use padded size when reading facets
- 689a38fd - Add nthreads parset key
- 1af15e3f - Fix type of retrieved parset values
- 10a33fc4 - Add tWGridderPredict in steps, rename tWGridderPredict in ddecal
- 6a22031a - Bump schaapcommon
Toggle commit list-
48e0cda9...c62e00da - 6 commits from branch
added 1 commit
- 13adeeca - Restore default direction for modeldata columns
added 1 commit
- 66db86fb - Use H5ParmPredict instead of wsclean in integration test
added 1 commit
- f87ccc89 - Use H5ParmPredict instead of wsclean in integration test
added 1 commit
- 3cbeb1ba - Fix WGridderPredict spectral integration test
- Resolved by Tammo Jan Dijkema
- Resolved by Tammo Jan Dijkema
- Resolved by Tammo Jan Dijkema
mentioned in merge request !1304 (merged)
requested review from @mnijhuis-tos