Skip to content

MPS compatibility on Apple Silicon#1003

Merged
carsen-stringer merged 2 commits intoMouseLand:mainfrom
OratHelm:main
Aug 24, 2024
Merged

MPS compatibility on Apple Silicon#1003
carsen-stringer merged 2 commits intoMouseLand:mainfrom
OratHelm:main

Conversation

@OratHelm
Copy link
Copy Markdown
Contributor

MPS support on Apple Silicon to enable use of the GPU. Works also with the GUI.
Training works on GPU with Pytorch 2.4.0.

OratHelm added 2 commits July 19, 2024 17:42
From what I've tested, it works well using the GPU (or with the CPU too, it's your choice), from the GUI or from a Python script.
However, I couldn't get the training to work, unfortunately. There are no errors but the generated model doesn't find any cells, and at all epochs train_loss=nan, test_loss=0.0000, except when using the GUI where train_loss at epoch 0 is non-zero.
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 23, 2024

Codecov Report

Attention: Patch coverage is 46.66667% with 24 lines in your changes missing coverage. Please review.

Project coverage is 55.97%. Comparing base (21789ec) to head (6b7fbc7).
Report is 18 commits behind head on main.

Files Patch % Lines
cellpose/core.py 48.14% 14 Missing ⚠️
cellpose/dynamics.py 33.33% 6 Missing ⚠️
cellpose/denoise.py 0.00% 2 Missing ⚠️
cellpose/models.py 80.00% 1 Missing ⚠️
cellpose/train.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1003      +/-   ##
==========================================
- Coverage   56.03%   55.97%   -0.06%     
==========================================
  Files          17       17              
  Lines        3876     3898      +22     
==========================================
+ Hits         2172     2182      +10     
- Misses       1704     1716      +12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@carsen-stringer carsen-stringer merged commit dc3848d into MouseLand:main Aug 24, 2024
@carsen-stringer
Copy link
Copy Markdown
Member

thank you!

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