Skip to content

Bugfix: Removing unused 'ARBITRARY_SIZE'#1082

Merged
carsen-stringer merged 3 commits intoMouseLand:mainfrom
Pablo1990:patch-1
Feb 7, 2025
Merged

Bugfix: Removing unused 'ARBITRARY_SIZE'#1082
carsen-stringer merged 3 commits intoMouseLand:mainfrom
Pablo1990:patch-1

Conversation

@Pablo1990
Copy link
Copy Markdown
Contributor

Found error:
Traceback (most recent call last):
File "/home/pablo/mambaforge/envs/colab_cellpose/lib/python3.10/site-packages/cellpose/export.py", line 56, in
from bioimageio.spec.model.v0_5 import (
ImportError: cannot import name 'ARBITRARY_SIZE' from 'bioimageio.spec.model.v0_5' (/home/pablo/mambaforge/envs/colab_cellpose/lib/python3.10/site-packages/bioimageio/spec/model/v0_5.py)

It turns out, they removed 'ARBITRARY_SIZE':
bioimage-io/spec-bioimage-io@39d3436

By removing the parameter, this code will work again.

Found error:
Traceback (most recent call last):
  File "/home/pablo/mambaforge/envs/colab_cellpose/lib/python3.10/site-packages/cellpose/export.py", line 56, in <module>
    from bioimageio.spec.model.v0_5 import (
ImportError: cannot import name 'ARBITRARY_SIZE' from 'bioimageio.spec.model.v0_5' (/home/pablo/mambaforge/envs/colab_cellpose/lib/python3.10/site-packages/bioimageio/spec/model/v0_5.py)

It turns out, they removed 'ARBITRARY_SIZE':
bioimage-io/spec-bioimage-io@39d3436

By removing the parameter, this code will work again.
@Pablo1990
Copy link
Copy Markdown
Contributor Author

Pablo1990 commented Jan 7, 2025

It now gives:

Traceback (most recent call last):
File "/home/pablo/mambaforge/envs/colab_cellpose/lib/python3.10/site-packages/cellpose/export.py", line 404, in
main()
File "/home/pablo/mambaforge/envs/colab_cellpose/lib/python3.10/site-packages/cellpose/export.py", line 361, in main
descr_input = descr_gen_input(path_test_input)
File "/home/pablo/mambaforge/envs/colab_cellpose/lib/python3.10/site-packages/cellpose/export.py", line 137, in descr_gen_input
SpaceInputAxis(id=AxisId("z"), size=ARBITRARY_SIZE),
NameError: name 'ARBITRARY_SIZE' is not defined

Edit: The next commit seems to fix it

@carsen-stringer
Copy link
Copy Markdown
Member

thanks!

@carsen-stringer carsen-stringer merged commit 20504f1 into MouseLand:main Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants