-
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
FileNotFoundError in test_perceptual_loss #7382
Comments
The cause of the error appears to be that the permission of the link has changed. MONAI/monai/losses/perceptual.py Line 257 in e1ffa7e
Hi @marksgraham and @Warvito, would you mind taking a look at this one? Is there any reason you have done this or has it been done accidentally? Thanks! |
May not be due to permission but the issue described here. |
I can now download so I think the problem must be hitting a daily download limit. I think we need to move to a non g-drive solution for hosting these files. How is this handled in the rest of MONAI? |
It may be about a daily download limit. We can put some small files here for test. |
Signed-off-by: YunLiu <55491388+KumoLiu@users.noreply.github.com>
Workaround for #7382 #7383 ### Description Based on the comment [here](wkentaro/gdown#291 (comment)), pin the gdown version as a workaround. Will review this one once gdown has some update internal. ### Types of changes <!--- Put an `x` in all the boxes that apply, and remove the not applicable items --> - [x] Non-breaking change (fix or new feature that would not break existing functionality). - [ ] Breaking change (fix or new feature that would cause existing functionality to change). - [ ] New tests added to cover the changes. - [ ] Integration tests passed locally by running `./runtests.sh -f -u --net --coverage`. - [ ] Quick tests passed locally by running `./runtests.sh --quick --unittests --disttests`. - [ ] In-line docstrings updated. - [ ] Documentation updated, tested `make html` command in the `docs/` folder. Signed-off-by: YunLiu <55491388+KumoLiu@users.noreply.github.com>
This issue is either sorted with the gdown fix or it's a matter of moving temp files to the extra test data repo so I'd close this for now. |
Workaround for Project-MONAI#7382 Project-MONAI#7383 ### Description Based on the comment [here](wkentaro/gdown#291 (comment)), pin the gdown version as a workaround. Will review this one once gdown has some update internal. ### Types of changes <!--- Put an `x` in all the boxes that apply, and remove the not applicable items --> - [x] Non-breaking change (fix or new feature that would not break existing functionality). - [ ] Breaking change (fix or new feature that would cause existing functionality to change). - [ ] New tests added to cover the changes. - [ ] Integration tests passed locally by running `./runtests.sh -f -u --net --coverage`. - [ ] Quick tests passed locally by running `./runtests.sh --quick --unittests --disttests`. - [ ] In-line docstrings updated. - [ ] Documentation updated, tested `make html` command in the `docs/` folder. Signed-off-by: YunLiu <55491388+KumoLiu@users.noreply.github.com> Signed-off-by: Mark Graham <markgraham539@gmail.com>
Workaround for Project-MONAI#7382 Project-MONAI#7383 ### Description Based on the comment [here](wkentaro/gdown#291 (comment)), pin the gdown version as a workaround. Will review this one once gdown has some update internal. ### Types of changes <!--- Put an `x` in all the boxes that apply, and remove the not applicable items --> - [x] Non-breaking change (fix or new feature that would not break existing functionality). - [ ] Breaking change (fix or new feature that would cause existing functionality to change). - [ ] New tests added to cover the changes. - [ ] Integration tests passed locally by running `./runtests.sh -f -u --net --coverage`. - [ ] Quick tests passed locally by running `./runtests.sh --quick --unittests --disttests`. - [ ] In-line docstrings updated. - [ ] Documentation updated, tested `make html` command in the `docs/` folder. Signed-off-by: YunLiu <55491388+KumoLiu@users.noreply.github.com> Signed-off-by: Juan Pablo de la Cruz Gutiérrez <juampatronics@gmail.com>
Workaround for Project-MONAI#7382 Project-MONAI#7383 ### Description Based on the comment [here](wkentaro/gdown#291 (comment)), pin the gdown version as a workaround. Will review this one once gdown has some update internal. ### Types of changes <!--- Put an `x` in all the boxes that apply, and remove the not applicable items --> - [x] Non-breaking change (fix or new feature that would not break existing functionality). - [ ] Breaking change (fix or new feature that would cause existing functionality to change). - [ ] New tests added to cover the changes. - [ ] Integration tests passed locally by running `./runtests.sh -f -u --net --coverage`. - [ ] Quick tests passed locally by running `./runtests.sh --quick --unittests --disttests`. - [ ] In-line docstrings updated. - [ ] Documentation updated, tested `make html` command in the `docs/` folder. Signed-off-by: YunLiu <55491388+KumoLiu@users.noreply.github.com> Signed-off-by: Yu0610 <612410030@alum.ccu.edu.tw>
The text was updated successfully, but these errors were encountered: