-
Notifications
You must be signed in to change notification settings - Fork 196
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 SW_FastEdit code #1689
Add SW_FastEdit code #1689
Conversation
Signed-off-by: Matthias Hadlich <[email protected]>
Can you fix the precommit checks.. you can run locally to identify the issues.. Please check the contrib guidelines to know the command to run the same locally |
Sure! The code also still contains some debug code which I will remove as well.
…On May 13, 2024 12:04:23 AM GMT+02:00, SACHIDANAND ALLE ***@***.***> wrote:
Can you fix the precommit checks.. you can run locally to identify the issues..
Please check the contrib guidelines to know the command to run the same locally
--
Reply to this email directly or view it on GitHub:
#1689 (comment)
You are receiving this because you authored the thread.
Message ID: ***@***.***>
|
for more information, see https://pre-commit.ci Signed-off-by: Matthias Hadlich <[email protected]>
Signed-off-by: Matthias Hadlich <[email protected]>
Signed-off-by: Matthias Hadlich <[email protected]>
@SachidanandAlle Should work with this commit. If you want to try it yourself, use the following weights file https://drive.google.com/file/d/1p59s-ehEyp39vlqULAd1eB8gV4hkmxR0/view?usp=drive_link Please tell me if there is anything else you need. |
Assigning to @diazandr3s to verify and merge |
Signed-off-by: Andres Diaz-Pinto <[email protected]>
Hi @matt3o, Many thanks for your contribution and sorry I just came back to you. Just to confirm, this model doesn't have the training class, right? can you please comment more on how you trained the model? I've uploaded the model to the MONAI Label repository - now users should be able to run inference as the other radiology models. Here is the link to the model for your reference: https://github.com/Project-MONAI/MONAILabel/releases/download/pretrained/radiology_segmentation_sw_fastedit_pet.pt I've also made a video shwoing how this model works using a sample PET I have: fastedit_pet.mp4Many thanks again! |
@diazandr3s Very nice, thanks for the integration :) Sadly I did not find the time to implement the training part. Please ping if someone is asking for it specifically and I'll see if I can get it to work. I think the greatest benefit of the model is specifically the non-interactive pretraining part - the rest is only slightly different from DeepEdit. About the training: 400 epochs of non-interactive pretraining on all AutoPET samples each epoch, so still with all three channels, but two of them are just zeroed out. Then 200 epochs of interactive training using the DeepEdit interactive sampling of clicks, but in very revamped manner - a lot faster and with some improvements. |
See this discussion for details, also @diazandr3s
Paper link
Link to the full code base.
Current limitations:
(where do can upload the weights?)