Closed bioimageiobot closed 8 months ago
@esgomezm why does the zenodo record also include the packaged bioimage-model.zip? Looks like it contains the same files as are in the record. This file duplication makes it unclear which ones were tested, etc.. I'm going to ignore the bioimageio-model.zip as our convention is to upload the files individually to zenodo to allow selective download.
back to the real issue:
The error suggests that it is a pytorch bug:
terminate called after throwing an instance of 'c10::Error'
what(): isTuple()INTERNAL ASSERT FAILED at "/opt/conda/conda-bld/pytorch_1640811723911/work/aten/src/ATen/core/ivalue_inl.h":1400, please report a bug to PyTorch. Expected Tuple but got String
Exception raised from toTuple at /opt/conda/conda-bld/pytorch_1640811723911/work/aten/src/ATen/core/ivalue_inl.h:1400
@esgomezm could you please post your working, local environment here for comparison? (mamba list -r
)
Hi hi, The zip file contains the same files that are individually uploaded. It's a silly duplication but it is a way for me to have a plan B in any course whenever the bioengine or the zoo stops working.
Here it goes:
ah, well. A trivial issue!
The collection's default pytorch version is 1.10 (which was resolved to 1.10.2), but you have 2.1.0 in your env...
The default has been used as pytorch_version
is not specified in the rdf.yaml. (We don't have a default in spec, but currently do allow to not specify the pytorch version. So for practical purposes the collection repo had to choose a default).
The next minor bioimageio.spec release will make pytorch_version
a mandatory field which will prevent this issue in the future.
thanks for sharing the env @esgomezm . I'll patch the rdf.yaml here in the collection repo...
Hi @FynnBe! Ok, then I'll update also the export to specify the torch version. I thought that the library was automatically testing what's the installed version. Now I see that there's a different error :D
Hi @FynnBe! Ok, then I'll update also the export to specify the torch version. I thought that the library was automatically testing what's the installed version. Now I see that there's a different error :D
my bad, I messed up the YAML syntax of the patch...
I'm excited about seeing this one becoming green! I think this will never change xD
passed!!!
This is an automatic PR created by the @bioimageiobot regarding changes to the resource item 10.5281/zenodo.10366411. 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: @esgomezm
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
.Keep proposed version(s) (and this resource in general if it is new) as pending: Close this PR without merging.
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: https://github.com/bioimage-io/collection-bioimage-io/pull/695