Omar Shrit

@rcurtin it was just a couple of deprecated flags that I have added that caused the issue. I was able to look inside the CMakeError it was nothing related to...

@rcurtin I removed the CheckAtomic restriction, now it is back to how it was, and it is working on normal g++ crosscompilers. I think we can merge it.

Thank you very much for adding this, it was on my list a long time ago and it is nice to see that you took care of it. Could you...

@marlamb please do not use force-push, otherwise all the history of the commits is gone and we need to start the review process from scratch The correct way of doing...

@TirelessClock could you fix the compilation error please ? we should be able to compile it locally.

this is too far from being merged. just a new structure

so if I understand correctly you want only to have the headers moved out to a new directory called include?

it should be in the above directory as mlpack.hpp and the `mlpack` directory should be installed and you PC and not only the internal of `mlpack/` directory

@zoq anything we need to do for this ?

@MarkFischinger looks good to me, I will approve it after the CI checks out. In the meantime, Would you please show the performance on another test with a higher order...