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

Some significant chemical clusters omitted in impact plots #20

Open
JG1ANDONLY opened this issue Apr 10, 2024 · 4 comments
Open

Some significant chemical clusters omitted in impact plots #20

JG1ANDONLY opened this issue Apr 10, 2024 · 4 comments

Comments

@JG1ANDONLY
Copy link

JG1ANDONLY commented Apr 10, 2024

I noticed there were fewer significant (p-values smaller than 0.05) clusters displayed in the impact plot than those in the excel file. For example, in my analysis, there are 25 significant clusters in the generated excel file, but there are only 21 clusters in the impact plot displayed. Is there a specific standard for dropping certain clusters besides looking at p-values only?

@barupal
Copy link
Owner

barupal commented Apr 10, 2024

Which chemrich function are you using ?

@JG1ANDONLY
Copy link
Author

Which chemrich function are you using ?

I used run_chemrich_chemical_classes() on t-test results, inputting compound name, smiles, p-values, effect-size, and sub-pathway information as 'set'.

@barupal
Copy link
Owner

barupal commented Apr 10, 2024

This line has the filtering criteria :
clustdf.alt.impact <- clustdf[which(clustdf$pvalues<0.05 & clustdf$csize>1 & clustdf$alteredMetabolites>1) ,]

cluster p-value should be < 0.05, cluster size should be at least 2 and significant metabolites within a cluster should be at least 2. This might explain the difference.

@JG1ANDONLY
Copy link
Author

JG1ANDONLY commented Apr 10, 2024

I see. But I think for some reason clusters whose p-values = 0.05 were also included in my result. Thank you!

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

2 participants