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

Add feature to print time taken at each step #40

Closed
sandeeprepakula opened this issue May 18, 2018 · 4 comments
Closed

Add feature to print time taken at each step #40

sandeeprepakula opened this issue May 18, 2018 · 4 comments

Comments

@sandeeprepakula
Copy link

sandeeprepakula commented May 18, 2018

Request to enhance the code for printing time taken for each of the steps like building thumbs, making 2D projections, computing clusters, etc.

Building 2D projections is taking for ever in my case with 300 clusters and 80K images. Any reason why?

@duhaime
Copy link
Member

duhaime commented May 25, 2018

@sandeeprepakula Do you just want to know the cumulative time consumed during each step of processing? Would it be sufficient to add timestamps in each of the log statements displayed on the console?

Can you say more about your system hardware? How many cores are you running on, and how much RAM do you have? KMeans (the clustering approach we use) is one of the fastest clustering techniques, but if you are out of RAM and are swapping, that will slow things down very quickly.

Do you have access to a machine with beefier hardware, such as a university research cluster or a sizeable AWS instance? If so, the easiest solution might be to leverage one of those hardware setups...

@sandeeprepakula
Copy link
Author

Yes, I do use AWS for my requirement. Just printing the timestamp would also help into the logs or to the terminal as an stdout.

@saketkc
Copy link
Contributor

saketkc commented Jul 5, 2018

This is partially addressed by #59

@duhaime
Copy link
Member

duhaime commented Jul 9, 2021

We're now printing timestamps at the start of each step, which should take care of this. If you'd like to revisit this idea, please reopen this issue!

@duhaime duhaime closed this as completed Jul 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants