Skip to content

BMDeep tutorial added #95

Open
DanielaSchacherer wants to merge 23 commits intoImagingDataCommons:masterfrom
DanielaSchacherer:bmdeep_tutorial
Open

BMDeep tutorial added #95
DanielaSchacherer wants to merge 23 commits intoImagingDataCommons:masterfrom
DanielaSchacherer:bmdeep_tutorial

Conversation

@DanielaSchacherer
Copy link
Contributor

As discussed, I added the BMDeep tutorial. Happy for feedback! :)

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@DanielaSchacherer
Copy link
Contributor Author

@fedorov did we already talk about this? If not, maybe we should briefly in the Slim meeting later today!

@fedorov fedorov requested a review from Copilot January 22, 2026 14:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@fedorov
Copy link
Member

fedorov commented Jan 22, 2026

I admit I lost track of this PR...

@DanielaSchacherer
Copy link
Contributor Author

no problem, me too :D

@fedorov
Copy link
Member

fedorov commented Feb 2, 2026

@DanielaSchacherer I pushed some minor changes, and also left some suggestion in the comments here: https://colab.research.google.com/drive/1kD_mEbfi1ozhyyS0WK9zl3vyKYMJJJjA?usp=sharing

@DanielaSchacherer
Copy link
Contributor Author

I took the feedback in and answered to your comments in the notebook linked above. I already changed the code to use ann_index, so let's not merge yet, but let me test as soon as ann_index is out :)

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