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.

Sign up for GitHub

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

Jump to bottom

swin_unetr_btcv_segmentation_3d: pre trained model download link broken #1672

Closed
dyollb opened this issue Mar 20, 2024 · 4 comments · Fixed by #1674
Closed

swin_unetr_btcv_segmentation_3d: pre trained model download link broken #1672

dyollb opened this issue Mar 20, 2024 · 4 comments · Fixed by #1674

Comments

@dyollb
Copy link

dyollb commented Mar 20, 2024 •
edited
Loading

Describe the bug

In the swin_unetr_btcv_segmentation_3d there are some links to download the pre-trained model. Unfortunately, these links are broken.

To Reproduce
Steps to reproduce the behavior:

  1. Go to swin_unetr_btcv_segmentation_3d
  2. Check the download for the pre-trained model (including wget commands)

Expected behavior
The download links are updated.

Comment
It seems some weights can be downloaded in the research contribution repo:
@KumoLiu
Copy link
Contributor

KumoLiu commented Mar 20, 2024

Hi @dyollb, thanks for your interest here.

The link works for me. Could you please try:
wget -O ./dataset_0.json 'https://drive.google.com/uc?export=download&id=1qcGh41p-rI3H_sQ0JwOAhNiQSXriQqGi'
Perhaps the reason is that you don't have a folder named data.

Thanks.

@dyollb
Copy link
Author

dyollb commented Mar 20, 2024

As I wrote in my report I am talking about the model weights, e.g.

image

and here

image

@KumoLiu
Copy link
Contributor

KumoLiu commented Mar 20, 2024

Apologies for my previous oversight. I've tried it again and the link for the weights can indeed be downloaded without any issue.

@dyollb
Copy link
Author

dyollb commented Mar 20, 2024

My bad: seems I copied the wget url incorrectly, but the other link is broken...

bad_link_monai_tutorial

KumoLiu added a commit to KumoLiu/tutorials that referenced this issue Mar 21, 2024
@KumoLiu
fix Project-MONAI#1672
941fc3e 
Signed-off-by: YunLiu <55491388+KumoLiu@users.noreply.github.com>
@KumoLiu KumoLiu mentioned this issue Mar 21, 2024
Merged
5 tasks
ericspod pushed a commit that referenced this issue Mar 21, 2024
@KumoLiu
Fix broken link in swin_unetr_btcv_segmentation_3d (#1674)
3a016a0 
Fixes #1672


### Checks
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [ ] Avoid including large-size files in the PR.
- [ ] Clean up long text outputs from code cells in the notebook.
- [ ] For security purposes, please check the contents and remove any
sensitive info such as user names and private key.
- [ ] Ensure (1) hyperlinks and markdown anchors are working (2) use
relative paths for tutorial repo files (3) put figure and graphs in the
`./figure` folder
- [ ] Notebook runs automatically `./runner.sh -t <path to .ipynb file>`

Signed-off-by: YunLiu <55491388+KumoLiu@users.noreply.github.com>
boneseva pushed a commit to boneseva/MONAI-tutorials that referenced this issue Apr 21, 2024
@KumoLiu
Fix broken link in swin_unetr_btcv_segmentation_3d (Project-MONAI#1674)
b663649 
Fixes Project-MONAI#1672


### Checks
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [ ] Avoid including large-size files in the PR.
- [ ] Clean up long text outputs from code cells in the notebook.
- [ ] For security purposes, please check the contents and remove any
sensitive info such as user names and private key.
- [ ] Ensure (1) hyperlinks and markdown anchors are working (2) use
relative paths for tutorial repo files (3) put figure and graphs in the
`./figure` folder
- [ ] Notebook runs automatically `./runner.sh -t <path to .ipynb file>`

Signed-off-by: YunLiu <55491388+KumoLiu@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix broken link in swin_unetr_btcv_segmentation_3d KumoLiu/tutorials
2 participants
@dyollb @KumoLiu