Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
Already on GitHub? Sign in to your account
RunningDeepZoomTiler not working properly with multiple workers #183
Comments
|
Hi @erexhepa, could you provide more details? Are you using |
erexhepa
commented
Oct 3, 2016
|
Hi @bgilbert, I'm indeed using the DeepZoomGenerator in the deepzoom_tile.py code which is then called from the command line. The tiling code goes through DeepZoomStaticTiler which is first called (run method) and then the tiling process is handled from inside the DeepZoomStaticTiler class. It's the same code as the one in the git repo. I can't remember the exception message because I'm running some jobs at the moment (calling several tiling processes manually through the command line works and is an alternative to the issue above) and I'm avraid will interfere with the ongoing analysis. It was an IO error raised by the TIFF library used to read the tiff inside the hierarchical structure of TIFF inside the SVS. Thanks for your help |
erexhepa
commented
Oct 3, 2016
|
https://github.com/openslide/openslide-python/blob/master/examples/deepzoom/deepzoom_tile.py this is the version of deepzoom I'm running with a few modifications in the main to be manage the multiple threads called at the command line. Elton |
|
Okay. When you have it, please post the exception message and traceback, as well as the command line you used to run |
erexhepa
commented
Oct 8, 2016
•
|
Hi, here is the error reproduced with the unmodified version of the deepzoom_tile.py
Elton |
RandomForest5a
commented
Mar 2, 2017
|
hey have u figure it out? |
erexhepa
commented
Mar 2, 2017
|
nope I couldn't find why it does that but I finally decided to handle the parallelization myself by using multiple threads. This way it works fine. |
RandomForest5a
commented
Mar 2, 2017
|
Me too.
Here is what I did. I modify the source code in openslide, shut down the
error arising code. It seems fine with only showing warning.
Cheers.
…
|
erexhepa
commented
Mar 3, 2017
|
Yes but be careful ! My understanding is that the error rises due to parallel processes (TO BE VERIFIED) extracting tiles at different resolution depths but effectively working on the same resolution, so one of the processes goes out of bound with regard to the X,Y coordinates. I didn't dig more because the parallelisation via multithreating of several slides at the same time (rather then multithreading of one slide) seems to work and the decompositions are also correct. |
RandomForest5a
commented
Mar 3, 2017
|
I gonna think about it carefully cheers!
…
|
erexhepa commentedSep 29, 2016
Hi,
I was experiencing some problems with DeepZoomTiler python wrapper. I have installed the python package of openslide and running it from Ubuntu16.
Not sure if it is an issue related to a digital pathology particular format, in this case SVS. What happens is that when in multithread mode with multiple workers; some tiles are not generated the workers and the workers hang up. There is an IO error exception raised by the tile reader which if catched and ignored you can get all thread finish but you miss a lot of tiles more or less randomy distributed in the zoom levels.
If you run with only one worker you manage to produce all tiles correctly. Just wanted to raise the issue in case it wasn't known but otherwise was wondering if someone has found a solution ?
Elton