Maik Nijhuis authored 3 years ago and Loris Lucido committed 3 years ago

- Output JUNIT XML file for each unit test.
- Discard existing Test.XML, which has a wrong format.
- Add junit-merge.py for merging JUNIT files.

Bram Veenboer authored 3 years ago and Loris Lucido committed 3 years ago

Index the subgrids as subgrid[2][s][y][x] instead of subgrid[s][2][y][x]
in the gridder kernel. Other kernels can now use subgrids[s]pol][y][x]
as they used to, simplifying the code.

Bram Veenboer authored 3 years ago and Loris Lucido committed 3 years ago

For now, only the gridder, degridder, adder and splitter kernels have
been updated. It is now mandatory to provide the nr_correlations
argument. The flop and byte computation has been updated as well.

Bram Veenboer authored 3 years ago and Loris Lucido committed 3 years ago

Replace the occurrences of this constant throughout the code.

Bram Veenboer authored 3 years ago and Loris Lucido committed 3 years ago

The gridder seems fine, the degridder produces only half of the
visibilities and these are wrong.

Bram Veenboer authored 3 years ago and Loris Lucido committed 3 years ago

It is still used in the OpenCL proxy, but since this Proxy is not really
used in practice, changing that code is still ToDo. The definition is
moved to idg-api, since it still uses Visibility.

Bram Veenboer authored 3 years ago and Loris Lucido committed 3 years ago

This is a workaround for runtime undefined reference issues when running
DP3.

Bram Veenboer authored 3 years ago and Loris Lucido committed 3 years ago

Use std::complex<float> instead, this requires upgrading of some
Array4D's to Array5D.

Bram Veenboer authored 3 years ago and Loris Lucido committed 3 years ago

This issue was introduced when the interface to sizeof_visibilities was
changed (from three to four arguments) and caused DP3 to fail.

Bram Veenboer authored 3 years ago and Loris Lucido committed 3 years ago

These now also use an Array4D<std::complex<float>> for visibilities.

Bram Veenboer authored 3 years ago and Loris Lucido committed 3 years ago

This is to prepare for imaging with nr_correlations different than 4,
which is hardcoded in the idg::Visibility type.