-
Notifications
You must be signed in to change notification settings - Fork 9
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
Update TAGAN-axons #614
Update TAGAN-axons #614
Conversation
Hi @FynnBe you changed the requirements file regarding the input numpy but the CI keeps failing. Do you know why? Is there a way we can help to correct it? |
I changed test-input.npy to test-input-cpu.npy, because "test-input.npy" is not present in the record, while "test-input-cpu.npy" is... The error "Could not run 'aten::empty_strided' with arguments from the 'CUDA' backend." looks like the something is hardcoded to live in GPU memory. The files gpu.pt, cpu.pt and TAGAN_Axons_G.pth are not referenced in the rdf.yaml at all, which is a bit confusing. |
The model format needs some changes to pass the CI.
Hi @cbouchard22 and @FynnBe The bad news: this gan is producing a different output each time, so we cannot test that the output test image provided is the same as the one that the network outputs when we test it. We need to discuss this (I'm opening an issue) and update the CI. Meanwhile, @cbouchard22 could you please upload the model corrected and update it? Please, find here the corrected model: https://drive.google.com/file/d/1ujdaAZwXFRYofxyoDrpX0kD6HgsO1TM9/view?usp=sharing |
This is an automatic PR created by the @bioimageiobot regarding changes to the resource item 10.5281/zenodo.7872357.
The following version(s) will be added:
Please review the changes and make sure the new item or version(s) pass the following check list:
Maintainers: @cbouchard22
Note: If you updated or re-uploaded another version for the current item on Zenodo, this PR won't be changed automatically. To proceed, you can do the following:
status
field asaccepted
, but change thestatus
under the current version toblocked
.Then wait for the CI on the main branch to complete. It should detect the new version(s) and create another PR for the new version(s).
Previous PRs of this resource: #601