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

Classifier dies when processing a larger data set #74

Closed
kamilzyla opened this issue Jul 7, 2020 · 6 comments
Closed

Classifier dies when processing a larger data set #74

kamilzyla opened this issue Jul 7, 2020 · 6 comments
Assignees

Comments

@kamilzyla
Copy link
Collaborator

When running the classifier on a "small sample of data" provided by Robbie (628 images), the process runs quite slowly and dies on the second batch. I was using my work laptop with 16 GB of RAM. @marekrogala suggests the images might have unnecessarily large resolution and scaling them down could possibly help.

@kamilzyla
Copy link
Collaborator Author

We tried with @swiezew to reproduce the issue on his Mac, but the classification finished without issues. I need to investigate the issue on my laptop further.

@kamilzyla kamilzyla assigned kamilzyla and unassigned swiezew Jul 7, 2020
@marekrogala
Copy link
Contributor

@swiezew how long does it take on your machine? if we extrapolate to numbers of images that we expect in real usage, how long would the inference take? is it using GPU on your machine?

@swiezew
Copy link
Contributor

swiezew commented Jul 7, 2020

@marekrogala I've worked with the 628 images sample. I used my 2013 Mac's CPU (2,7GHz, 16GB RAM). Inference was divided into 10 batches, each took approximately 100 seconds (15 minutes in total). That means each image took around 0.7sec. The typical size of the data to be analysed has been said be 30000 images, which would take around 6hours on my laptop.

@kamilzyla
Copy link
Collaborator Author

It seems the crash is indeed due to the classifier running out of memory. I noticed that I am consistently able to process the full data set if I close all Chrome instances; with a few tabs open the classifier would crash on the 10th batch and with many tabs open it would crash on the 3rd.

Our users probably won't have more than 16 GB RAM on their machines (?), so solving this issue is a priority. Scaling the images down might suffice; I'll create a separate issue for it.

@marekrogala
Copy link
Contributor

marekrogala commented Jul 9, 2020 via email

@kamilzyla
Copy link
Collaborator Author

Closing - we didn't have any reports of this issue recently.

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

3 participants