-
Notifications
You must be signed in to change notification settings - Fork 9
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
cellfinder v1.3.0 blog post #177
Comments
Here are the graphs and correlation/coefficients comparing tensorflow to torch on both Brain 1 and Brain 2 from the paper: The timing information is below:
|
To clarify some points @IgorTatarnikov:
|
Correct, seems torch is a tiny bit better than TF. For detection timings, all were run on the original machine from the paper. The "paper" columns are numbers taken from the cellfinder paper, the "torch" columns are the means of two runs using the detection code that is in the current release but with a torch backend. The last column is combining the new detection from brainglobe/cellfinder#407 with a torch backend. |
So to summarise, the next release will be pretty much the same accuracy and speed (but easier to maintain for us)? |
Yup! It might be interesting to include some comparisons using a c-fos brain. I'm curious how the new release will compare to the old release with millions of cell candidates. |
@alessandrofelder do you happen to have any data like this to hand? I think we should include it if we have it, otherwise we can post something later on. This blog post could end up being longer than the original paper otherwise! |
Due to various updates to other packages, have decided to create a blog post for the new version of the metapackage. This blogpost can be solely about the move to torch. |
Yes, I'll DM @IgorTatarnikov the details to it on our internal storage |
The next version of cellfinder is likely to be a (relatively) large one, and we should write a blog post to outline the differences. This will include:
~.brainglobe/cellfinder
, how the old models can be deleted, and how to re-install them in the new location (Document thecellfinder_download
CLI that can be used to download the pre-trained model #176) (Simplify model download cellfinder#414)cc @alessandrofelder
The text was updated successfully, but these errors were encountered: