Vaibhav Thakkar

Hi @cchiu727, yes this issue is open. It would be great if we can first discuss some potential approaches and then go ahead with implementing it.

Any updates @cchiu727 ?

Hi @vissarion, I think this check should be automated so that this is ensured in every PR. I find this stackoverflow answer helpful: https://stackoverflow.com/a/57900525/7317290. @Neel-shah-29 can you check this out?

Hi @Neel-Shah-29, I would prefer that the automated check be done in this PR only, reason being that whenever a bug is solved, the corresponding PR should add some form...

Hi @Neel-Shah-29, if you see your log message no target is made for any of the header files. You would have to change the glob expression for searching files. I...

@Neel-Shah-29, I think that is because of only using filenames without extensions for creating the custom targets, you can also include the extension for naming the target. You will have...

Hi @Neel-Shah-29. I suspected this would happen as a single header file can include other header files. A fix for this can be to provide the path to essential header...

The issue is same here, you have to provide the include directories for these header files.

You can use CMAKE_SOURCE_DIR variable to set the absolute path of working directory.

Hi, I am just curious why do you want to replace the pointers? I am asking because firstly, Eigen doesn't provide begin() or end() methods for its matrix and vector...