Missing Header in IO/Reader.cpp

[diogosiebert]

I believe there is an include missing in the IO/Reader.cpp

#include "IO/silo.h"

Without this the commit 3289bf336b is not building with SILO support.

[JamesEMcClure]

That is good to know. I will check it out.

As a note, Silo is now an optional dependency and the I/O is fully functional with HDF5 alone. This was done because Silo development was lagging significantly behind HDF5, and we were unable to use newer versions of HDF5 without deprecating silo. It may be worthwhile to try the HDF5 only build, since this eliminates a dependency and should not limit what you can do with LBPM.

[pirouzs]

Hi I faced the same issue. It took me a few hours to realize that adding #include "IO/silo.h" to the "Reader.cpp" file might potentially resolve the problem. After making this adjustment, the build completed without any errors.