Skip to content
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

Merge Cellpose In To Main #36

Merged
merged 15 commits into from
Oct 31, 2024
Merged

Merge Cellpose In To Main #36

merged 15 commits into from
Oct 31, 2024

Conversation

SRodwin
Copy link
Collaborator

@SRodwin SRodwin commented Oct 23, 2024

No description provided.

@SRodwin SRodwin requested a review from krpandit October 23, 2024 18:31
@SRodwin SRodwin self-assigned this Oct 23, 2024
Copy link
Collaborator

@krpandit krpandit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I put in a few minor comments to change. I also tried running the tests but I got this error below before any tests could actually run. Any ideas why? The tests shouldn't import pytorch, but it should submit a job which does.

E ImportError: Failed to load PyTorch C extensions:
E It appears that PyTorch has loaded the torch/_C folder
E of the PyTorch repository rather than the C extensions which
E are expected in the torch._C namespace. This can occur when
E using the install workflow. e.g.
E $ python setup.py install && python -c "import torch"
E
E This error can generally be solved using the develop workflow
E $ python setup.py develop && python -c "import torch" # This should succeed
E or by running Python from a different directory.

@krpandit krpandit merged commit f4b6c6e into cellpose Oct 31, 2024
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