saezlab / visium_heart

Spatial transcriptomics of heart tissue
GNU General Public License v3.0
72 stars 36 forks source link

null for rownames(proj) #10

Open ellyhung opened 1 year ago

ellyhung commented 1 year ago

First of all, excellent work! Congratulations!

I have faced an issue trying to reproduce your analysis. In line 52 of script in snATAC_seq/data_integration)/05_clustering.ipynb, it seems that rownames(proj) is null as attched. 截圖 2023-03-22 下午11 26 06 I am wondering if I missed something here. I will be grateful for any response. Thank you in advance

lzj1769 commented 1 year ago

Hi @ellyhung

Can you check if the proj is loaded?

ellyhung commented 1 year ago

Hi Zhijian, Thanks for the reply. Yes it is successfully loaded.  The only difference from my analysis to yours is that I used my dataset instead. But it shouldn’t be an issue since I processed all the steps as you provided and the outputs are in similar form as you shown. I checked the rownames. @.@rownames shows the sample names; while @.@rownames shows all the cell (format: samplename#sequence). 

On Thursday, March 23, 2023, 12:02 AM, Zhijian Li @.***> wrote:

Hi @ellyhung

Can you check if the proj is loaded?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

lzj1769 commented 1 year ago

Hi @ellyhung

It's possible that ArchR updated the way it organized the data.

Anyway, here you can try an alternative way to get the cell names, but be careful that ArchR always adds the sample name to the barcode while Seurat doesn't.

ellyhung commented 1 year ago

Thank you so much for the information. Yes, that's possibly the reason.

In addition, I am wondering if I could know the reason that you remove cluster14, 22, 21, 26, 25 for annotation?

lzj1769 commented 1 year ago

Hi @ellyhung ,

We removed these clusters because we found they were driven mainly by artifacts such as low data quality, patient-specific factors, high doublet score, etc, compared with other clusters.

And we couldn't identify these cells without clustering results available.

ellyhung commented 1 year ago

Thank you so much for your information!