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

Include notebook showing the working principle #189

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

bputzeys
Copy link
Collaborator

of different embedding modes for single cell data

Copy link

@bputzeys bputzeys force-pushed the create-new-embedding-mode-notebook branch 4 times, most recently from 0b8bd96 to ffaa1cd Compare February 18, 2025 09:49
@giogix2
Copy link
Contributor

giogix2 commented Feb 18, 2025

When printing the results embeddings, it'd be useful to print also the array shape. This would help the user to verify that the output makes sense. For example, in cell mode, they could just observe whether the number of rows in the output arrays corresponds to the number of cells.
Just a minor suggestion, so feel free to ignore it.

@bputzeys bputzeys force-pushed the create-new-embedding-mode-notebook branch 2 times, most recently from 2024ac9 to 53f1db3 Compare February 18, 2025 12:26
of different embedding modes for single cell data
@bputzeys bputzeys force-pushed the create-new-embedding-mode-notebook branch from 53f1db3 to 33c3d53 Compare February 18, 2025 12:47
@bputzeys bputzeys merged commit cbc9b6d into main Feb 18, 2025
5 checks passed
@bputzeys bputzeys deleted the create-new-embedding-mode-notebook branch February 18, 2025 13:17
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