Tangram icon indicating copy to clipboard operation
Tangram copied to clipboard

why enforce lowercase?

Open giovp opened this issue 1 year ago • 4 comments

https://github.com/broadinstitute/Tangram/blob/3c87a25d49ad50b7f85ba68c67d31b46e38c3847/tangram/mapping_utils.py#L40-L41

think it'd be useful if this operation is optional.

giovp avatar Aug 08 '22 09:08 giovp

Hi @giovp ,

Thanks for your suggestion! The initial purpose of these two lines of commands are to get rid of the inconsistent capitalization of gene name. You are right, it is not always needed and thus it is better to make it optional.

I have just updated Tangram function pp_adatas with an additional parameter gene_to_lowercase. The default value is set as True. If not needed, people can alway set it as False to prevent conversion of the genes names to lower cases. One additional line of instructions is added before the pp_adatas function in tutorial.

Hejin0701 avatar Aug 15 '22 22:08 Hejin0701

thank you!

giovp avatar Aug 18 '22 18:08 giovp

Hi,

It looks like this enforcing lower case was not only in pp_adatas, but also in tg.project_genes. After running it my sc_adata genes become all lower case. Can you please also provide an option to disable it there?

Thanks a lot!

yinan-wan0 avatar Jan 26 '24 19:01 yinan-wan0

Indeed! that's really problematic and give rise to all sorts of issues, am reopening this cause the issue is not resolved

The initial purpose of these two lines of commands are to get rid of the inconsistent capitalization of gene name

I would also like to mention that there is no reason that such inconsistency should be resolved inside the method call, especially without throwing any warning.

giovp avatar Mar 13 '24 15:03 giovp