chanzuckerberg / single-cell-explorer

Hosted version of cellxgene
MIT License
11 stars 2 forks source link

BUG: Bug 1: Selected cells discrepancy between umap and spatial #936

Closed tihuan closed 5 months ago

tihuan commented 5 months ago

Context: Slack: https://czi-sci.slack.com/archives/C0247APK621/p1711647089759669

Couple things to iron out towards Visium support. They may be caused by an embedding having null values for some of the obs.

  1. umap shows 4992 of 4992 cells, but only 4113 cells are included in the obsm[‘X_umap’].

Toggling triggers/resolves

Repro: start on umap (bug1) --> spatial --> umap --> spatial (bug2 + bug3)

Example: https://cellxgene.cziscience.com/e/a3b219d6-6606-4d82-82fe-fe76f3889c3c.cxg/

Bug 1:

Current: umap shows 4992 of 4992 Fix: Should show 4113 of 4113

Screenshot 2024-03-28 at 11.34.22 AM.png

Private Zenhub Video

tihuan commented 5 months ago

This ticket is split out from #858

tihuan commented 5 months ago

Seve figured out a better solution, so I'll create a new PR for that

tihuan commented 5 months ago

PM for Timmy reference:

https://chanzuckerbergteam.slack.com/archives/DTLKZ6GDA/p1716330242831049