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

Some issues with display of rendering, etc. #36

Closed
SahPet opened this issue Jun 9, 2022 · 3 comments
Closed

Some issues with display of rendering, etc. #36

SahPet opened this issue Jun 9, 2022 · 3 comments
Labels
enhancement New feature or request
Projects

Comments

@SahPet
Copy link

SahPet commented Jun 9, 2022

I have briefly tested the new FastPathology version. More stable. but some issues:

  1. crashes when I try to load small tiff files, but not large ones. When I test prediction for mitoses they become invisible at "overview" ca. 4x magnification and thus it becomes almost pointless to run the algorithm when you have to look for all green dots. In Qupath, small annotations are magnified when you zoom out so they are still visible. Is this possible to get into FastPathology (that all annotations have a given line thickness regardless of magnification?)

  2. The NoCodeSeg import script does not work on the tiff file I got out in the results folder. Correct size, but too large for the import script (this was a standard WSI, but large about 60k x 100k pixels, and MIB/DeepMIB/Matlab struggles with these as well at 40x resolution).

  3. The import script for direct import from fast pathology at the NoCodeSeg page probably does not work either as it is a new folder structure with lots of subfolders. I would have dropped all these subfolders in results, or at least had a little less of them.

  4. It is difficult to know whether the algorithm is actually running or not - no progress bar and no time indication for estimated finished prediction. For mitoses, I waited 15 minutes and it was finished, but could just as easily have been nothing that happened - I did not know that as a user.

@andreped andreped added the enhancement New feature or request label Jun 9, 2022
@andreped
Copy link
Member

andreped commented Jun 9, 2022

Great suggestions, @SahPet!

Some of these suggestions, there already exists issues for. For 1) see #33, and 2) see #35, which I just created an issue for after discussing this with you on Teams.

Next time it is better to split sets of issues into individual issues on github.

That makes it easier for us to track, and we can more easily distribute the workload.

But great suggestions, we will try to solve all of them within a reasonable amount of time :)

@andreped andreped added this to To do in v1.1.0 via automation Jun 9, 2022
@SahPet
Copy link
Author

SahPet commented Sep 9, 2022

So, here is a visual demo of what the point:

Segmentation networks for small rare objects, such as mitoses, rare cell types, etc. are not visible on the overview, which makes its really hard to use FastPathology for this purpose:
Screenshot_291

.... even though there are quite a few there:
Screenshot_292

In QuPath for example, small objects are clearly visible at full overview as well as on full zoom, so should be possible, by keeping the border line thickness rendering independent on zoom level for example.

@andreped
Copy link
Member

Some of the points here has been adressed already. Whereas for those that remains, new targeted Issues have been created. Hence, this is now a duplicate. Closing this issue for now.

v1.1.0 automation moved this from To do to Done Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

No branches or pull requests

2 participants