The big dadaflow merge
Dadaflow is now separated from the rest of the cmake build in its own submodule. This is not enabled by BUILD_SUBMODULES but by a new option BUILD_DADAFLOW (defaulted to OFF).
A couple of concerns here:
- the inclusion of the mathdx dependency is not well handled at the moment.
- this changes the C++ language standard in cmake
- Additional warning have been enabled. The merge request includes some fixes to non-dadaflow parts of psrdada_cpp
Merge request reports
Activity
added Feature label
requested review from @twinchen
assigned to @ebarr
On the mathdx side I note that Bram over in Astron has written this: https://git.astron.nl/RD/mathdx
added 1 commit
- 6116b291 - added base class for resource allocator and sigproc unpacker tests
added 1 commit
- 2796d7fc - added explicit header set specialisation for strings
added 1 commit
- 4e2de1eb - updated to format epoch and picoseconds correctly for DSPSR
added 1 commit
- afced011 - Remove fftdetacc from dadaflow and its mathdx dependency
mentioned in commit ca9b4c9a
Please register or sign in to reply