Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Insufficient memory issue on windows #12

Open
sivavenkatay opened this issue Mar 12, 2013 · 8 comments
Open

Insufficient memory issue on windows #12

sivavenkatay opened this issue Mar 12, 2013 · 8 comments

Comments

@sivavenkatay
Copy link

I was successful in compiling the binary for windows (had to add export macros for the classes to generate the lib file).

But when I run the exe with a simple model, it runs until the convolution (and prints a convolution time) and then crashes with an error like this -

OpenCV Error: Insufficient memory (Failed to allocate 64244 bytes) in unknown fu
nction, file ......\src\opencv\modules\core\src\alloc.cpp, line 52

It fails with all images (small or large). Has anyone encountered this before?

@sivavenkatay
Copy link
Author

It's crashing here (In DynamicProgram.cpp)-

Ix.resize(nscales, vector3DMat(ncomponents));
Iy.resize(nscales, vector3DMat(ncomponents));
Ik.resize(nscales, vector3DMat(ncomponents));
rootv.resize(nscales, vectorMat(ncomponents));
-----> rooti.resize(nscales, vectorMat(ncomponents));

And I have plenty of memory on my machine.

@Answeror
Copy link

It consumes 2G memory on my win7 x64...
After I move the backtracking part into dp algorithm, it works. It consumes about 280M at most.

@bnascimento
Copy link

Could you provide the code?

@Answeror
Copy link

See my fork: Answeror@65001f6

@sivavenkatay
Copy link
Author

@bnascimento I am running the example code that comes with the project.

@Answeror - will try your fork and let you know.

@bnascimento
Copy link

@sivavenkatay you running a 32 bit or 64 bit implementation?
32bit has a memory limit of 2GB

@bnascimento
Copy link

@Answeror will test your code then let you have some feedback..
anyway ty

@vrabaud
Copy link
Member

vrabaud commented Mar 28, 2013

@Answeror should your function be merged upstream ? If so, please make a pull request.

Answeror added a commit to Answeror/PartsBasedDetector that referenced this issue Apr 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants