-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
WSIReader
support for reading by mpp or magnification
#4980
Comments
WSIReader
support for read by magnificationWSIReader
support for reading by magnification
@cooperlab what do you think? |
@drbeh LGTM. The guiding principles here are that developers shouldn't have to do a lot of calculations to determine what they have or what's been done to it, and to make clear what definitions like |
WSIReader
support for reading by magnificationWSIReader
support for reading by mpp or magnification
Hi @drbeh , as discussed in the MONAI meeting last Tuesday, I will have a look at this issue. In our group we solved a similar problem and this might work here as well! |
Hi @drbeh, sorry for the delay, but I am almost done with a first implementation. However, there are a few detailed questions I have to make sure that the function gives the desired output. Should I post them here, or should I ask them in the next monai core meeting? |
Hi @NikolasSchmitz, that is fantastic! It's best if you ask here to make sure that anyone with an answer can help and then followup in our next meeting. Thanks |
Okay, sure! Assuming the simple and straightforward solution to this matter, we would take the image at the next lower wsi level and then downscale it to a resolution that fits the user-provided mpp value.
Personally, I would prefer 3. with a default value for tolerances atol and rtol. If one of the wsi levels is within the user's mpp +/- tolerance, then we can safely select the corresponding wsi level. Otherwise, we can scale down from the next larger representation of the wsi. I attached three images showing three cases, where case 1 shows the simple and straightforward solution, and cases 2 and 3 show the above described issue without and with a tolerance range. I thought it might help to visualize the problem. My questions are now:
|
@NikolasSchmitz my suggestion re upscaling is that users should have to opt-in to enable this. It's uncommon but I really like the option to save compute if the next highest magnification is very large. |
@NikolasSchmitz Just to summarize what we discussed about your questions:
Please do not hesitate to reach out if you have any other questions and thanks for contributing to MONAI. |
Thanks for the helpful feedback, everyone! I will create the draft pull request in the next days and implement what we discussed. |
This feature was added in #7574 |
In MONAI Pathology WG, we have discussed about the need for being able to read whole slide images at specific resolution (mpp or magnification) in addition to reading at a specific level. The feature request has been made by @cooperlab in cucim repo rapidsai/cucim#389.
I have discussed with cuCIM team about this topic and the conclusion was that it may be more useful to implement this functionality in MONA rather than in cucim for few reasons:
To achieve this, the following functionalities need to be added to MOANI
WSIReader
:The text was updated successfully, but these errors were encountered: