PChambino / pulse

Android-based implementation of Eulerian Video Magnification for vital signs monitoring

Home Page:p.chambino.com/dissertation

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Opencv manager compat issue?

JontiH opened this issue · comments

Hi,

I tried to download your app from the play store and after downloading opencv manager as instructed I got a message that opencv had not been untialized correctly and the app would have to shut down.

Hi Jonathan! I think the OpenCV version of the app needs to be upgraded for some new phones. I started a opencv3 branch, but I haven't had the change to test it well enough. Would you be interested in testing it? I can build an apk or you can try to build it yourself, let me know which one you prefer.

@PChambino I tried building this and I see the below error:
-pre-build: [exec] Android NDK: WARNING:jni/Android.mk:pulse: non-system libraries in linker flags: -lopencv_java3 [exec] Android NDK: This is likely to result in incorrect builds. Try using LOCAL_STATIC_LIBRARIES [exec] Android NDK: or LOCAL_SHARED_LIBRARIES instead to list the library dependencies of the

Can you tell me what wrong with what Im doing?
Trying this on ubuntu 16.

I understand that the above is probably just a warning...This is what follows:
humb: pulse <= pt_chambino_p_pulse_Pulse_Face.cpp [armeabi-v7a] Compile++ thumb: pulse <= pt_chambino_p_pulse_Pulse.cpp [armeabi-v7a] Compile++ thumb: pulse <= Pulse.cpp In file included from jni/Pulse.cpp:5: In file included from jni/ext/opencv.hpp:6: jni/ext/../profiler/Profiler.h:138:40: error: unknown register name '%eax' in asm asm volatile("cpuid" : : : "%eax", "%ebx", "%ecx", "%edx" ); ^ jni/ext/../profiler/Profiler.h:148:36: error: invalid output constraint '=a' in asm asm volatile("rdtsc" : "=a" (__a), "=d" (__d)); ^ 2 errors generated.

@bmvbab i have the same problem , do u solve this issue ?

@khaledsayed hav'nt really solved it, I did not try that further.

@PChambino @khaledsayed did you guys find any solution? @PChambino are you planning to take this forward

Setting up opencv itself in not easy

@bmvbab have you got any solution to this problem?

commented

And i am failing with termux.

Sorry guys, had given up on it, did not try in a while.

I haven't had the chance to pick this up and it is not at the top of my priority list unfortunately. It should still work on older phones (4 years ago), but I still don't have an Android phone at the moment. Also OpenCV is already on version 4, so it is a bit difficult to keep up with an always changing library.