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

Questions about "Construct GEMs" #25

Open
wangzlgithub opened this issue Dec 26, 2024 · 3 comments
Open

Questions about "Construct GEMs" #25

wangzlgithub opened this issue Dec 26, 2024 · 3 comments

Comments

@wangzlgithub
Copy link

Hello, thanks for developing this fantastic tool.

I don't think the parameter layer_key play a role when the inputting data has been normailzed in Construct GEMs step:

fs.pp.construct_gems_using_pyliger(adata,
                                n_gems = 10,
                                layer_key = 'counts',
                                condition_key = condition_key)

Because I saw the source codes in flowsig/preprocessing/_gem_construction.py as below:

ad.X = csr_matrix(ad.layers[layer_key].copy())
ad.obs.index.name = 'index'
ad.var.index.name = 'index'
# Create LIGER object
adata_list = []
for cond in conditions:
adata_cond = adata[adata.obs[condition_key] == cond].copy()
adata_cond.uns['sample_name'] = cond
adata_list.append(adata_cond)
adata_liger = pyliger.create_liger(adata_list, make_sparse=True)

@axelalmet
Copy link
Owner

Hello wangzlgithub,

Wow, good catch! This explains why a previous issue had noticed a discrepancy between the GEMs I generated for the original paper generated by the current version of FlowSig. Thank you so much!

I will update FlowSig ASAP. The simple fix should be to replace line 27 with:
adata_cond = ad[ad.obs[condition_key] == cond].copy()

@zhangyl686
Copy link

Hello wangzlgithub,
I've been trying to install this package for half a month, but all attempts have failed. Every time I try to import this package in Python, it prompts me to install other dependent packages. After I install them, it prompts me to install yet another dependent package. After installing some dependent packages, conflicts arise among them. I've tried dozens of times. Could you please teach me how to install the flowsig package successfully?
Thank you very much indeed!
My email address is [email protected].

@tenlives
Copy link

tenlives commented Mar 1, 2025

Hello wangzlgithub, I've been trying to install this package for half a month, but all attempts have failed. Every time I try to import this package in Python, it prompts me to install other dependent packages. After I install them, it prompts me to install yet another dependent package. After installing some dependent packages, conflicts arise among them. I've tried dozens of times. Could you please teach me how to install the flowsig package successfully? Thank you very much indeed! My email address is [email protected].

I have also have trouble with installing this package according current install mehtods. Could you provide us a simple and successful way ? Thank you very much! @axelalmet

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

No branches or pull requests

4 participants