microsoft / yardl

Tooling for streaming instrument data
https://microsoft.github.io/yardl/
MIT License
29 stars 5 forks source link

xtensor required version undocumented #75

Closed KrisThielemans closed 8 months ago

KrisThielemans commented 9 months ago

For whatever reason, my conda install got xtensor=0.21.10. The generated code fails to compile though as it xtensor_container doesn't have the flat member. https://github.com/xtensor-stack/xtensor/commit/50e3d42c4316ff01c12d3e039db567b42f009772 says this means at least 0.23.10 is required.

Ideally, this minimum version should be added to the generated CMakeLists.txt.

Of course, the same holds for other dependencies.

johnstairs commented 8 months ago

Thanks for the feedback. Fixed now.