C++ optional build of HDF5/NDJSON support

microsoft / yardl

Tooling for streaming instrument data

https://microsoft.github.io/yardl/

MIT License

29 stars 5 forks source link

Open KrisThielemans opened 7 months ago

KrisThielemans commented 7 months ago

I suggest to add

option(${prefix}_HDF5_SUPPORT "Add HDF5 protocol" ON)

or similar, and also for NDJSON. Could be advanced options. This would allow the advanced to switch off something that they don't need.

johnstairs commented 7 months ago

That's a good idea. We could also allow opting out of generating the files in the first place in the _package.yml file.