Skip to content

Update runcellpose.py#304

Open
davewabbo wants to merge 1 commit into
CellProfiler:masterfrom
davewabbo:CP4-cellpose4_macos_unbound-workaround-fix
Open

Update runcellpose.py#304
davewabbo wants to merge 1 commit into
CellProfiler:masterfrom
davewabbo:CP4-cellpose4_macos_unbound-workaround-fix

Conversation

@davewabbo
Copy link
Copy Markdown

@davewabbo davewabbo commented May 13, 2026

Disclaimer: Chatgpt was used to add the patch!

Added re-try for "UnboundLocalError: local variable 'y_data' referenced before assignment"

This annoying error happens randomly with multiple workers and not specific to an invididual image to catch any that throw e.g. "index 634465 is out of bounds for axis 1 with size 2160 (size of image)" errors leading to unassigned y_data. Context: I'm running CP4-Cellpose in its own env (CP==4.2.6, cellpose v4) on MacOS M5max

@bethac07
Copy link
Copy Markdown
Member

Hi,

Thanks for your interest in contributing to CellProfiler!

This seems to be a rather large diff for what you describe as a small change. Can you limit your diff to only the necessary change? Also, the error described is not currently reported and would be unusual to happen stochastically- can you tell us a bit more and include a full stack trace?

@gnodar01
Copy link
Copy Markdown
Member

@davewabbo Just fyi, the reason the diff shows everything as deleted and replaced, even though not that many actual lines are different, is because chatgpt used different line endings (NL/0a instead of CRLF/0d0a). So git diff sees each line as different.

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.

3 participants