trishume / SmartGaze

WIP Open source IR eye tracker for the Eye Tribe tracker hardware
GNU General Public License v2.0
28 stars 13 forks source link

Problems compiling (OSX, latest Halide release) #5

Open baekgaard opened 7 years ago

baekgaard commented 7 years ago

I've tried to build the SmartGaze on OS-X after having installed halide with brew.

I get compile errors as follows:

.../src/SmartGaze/src/halideFuncs.cpp:81:15: error: cannot refer to class template 'Buffer' without a template argument list inParam.set(Buffer(UInt(m.elemSize()*8), m.cols, m.rows, 0, 0, m.ptr())); ^ /usr/local/include/Halide.h:1594:48: note: template is declared here template<typename T2, int D2> friend class Buffer; ^ /Users/pgba/src/SmartGaze/src/halideFuncs.cpp:84:37: error: expected parameter declarator Image output(Buffer(UInt(8), out.cols, out.rows, 0, 0, out.ptr())); ...

Looks like the Buffer type change into a template instead in the latest release of Halide?

Any recommendations as to the quickest path forwards?

trishume commented 7 years ago

One possibility is try to figure out the code and fix it, since SmartGaze is pretty useless unless you intend to modify the code to do something useful anyway.

Another is that I think the latest pipeline might not actually use Halide at all, it may just be left in in case I want to use it, you may try commenting out all the Halide code and it might still work.

baekgaard commented 7 years ago

One possibility is try to figure out the code and fix it, since SmartGaze is pretty useless unless ...

Just to close this "discussion": Not sure about the sarcastic tone and why you would assume I wasn't looking at the code? I just naively assumed you might have had some ideas of what changed with Halide since you have been using it (and I hadn't so far)...

In any case, I fixed the issue for the application I had in mind.

trishume commented 7 years ago

Looking back yah sorry I was overly snarky. I get a lot of people creating issues on this and eyeLike who expect to compile it and find some sort of fully featured eye tracker library that solves their issue without having to work on the code, which neither project is. In this case I read your issue totally uncharitably as being like that, and reading it again it didn't merit that response at all.

I'm interested to heard about what you want to use SmartGaze for if you can tell me. Let me know if there's anything I can help you with.

ChristianFrisson commented 6 years ago

Hi @baekgaard and @trishume,

I have just installed the latest halide version available as formula before the last update of SmartGaze and the latter would compile for me on macOS (10.6.9). brew uninstall halide; brew install https://raw.githubusercontent.com/halide/homebrew-halide/7c3a7653549470a04a3033464130751016fd0434/halide.rb

Best regards, Christian