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

[WIP] ProteoRE topGO enrichment analysis 20210412 #598

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

combesf
Copy link
Collaborator

@combesf combesf commented May 17, 2021

Another ProteoRE tool for review.
Thanks !

Copy link
Member

@bgruening bgruening left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@combesf I added a few comments. Most of them are also true for the other tools.

tools/proteore_topgo/.Rhistory Outdated Show resolved Hide resolved
tools/proteore_topgo/.shed.yml Outdated Show resolved Hide resolved
tools/proteore_topgo/topGO.xml Outdated Show resolved Hide resolved
tools/proteore_topgo/topGO.xml Outdated Show resolved Hide resolved
tools/proteore_topgo/topGO.xml Outdated Show resolved Hide resolved
tools/proteore_topgo/topGO.xml Show resolved Hide resolved
<option value="parentchild" >ParentChild</option>
</param>
<param name="threshold" type="text" label="p-value threshold (e.g : 1e-3)" value="1e-3"/>
<param name="correction" label="Multiple testing procedure (p-value adjustment)" type="select">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indentation ... I recommend using https://github.com/galaxyproject/galaxy-language-server it will format your files nicely and will point you to a few limiting issues.

<param name="genelist" value="ID_Converter_FKW_Lacombe_et_al_2017_OK.txt"/>
<param name="column" value="c8"/>
<param name="header" value="TRUE"/>
</conditional>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add a test for at least every conditional. The Galaxy language server has a feature to bootstrap the test cases.

tools/proteore_topgo/topGO.xml Outdated Show resolved Hide resolved

**Galaxy integration**

Lisa Perus, Lien Nguyen, Florence Combes, Yves Vandenbrouck - CEA, INSERM, CNRS, Grenoble-Alpes University, BIG Institute, FR
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you add this as proper citation? maybe in bibtex?

@bgruening
Copy link
Member

@combesf can I help here in some way?

@combesf combesf changed the title [WIP] ProteoRE topGO enrichment analysis 20210412 ProteoRE topGO enrichment analysis 20210412 May 31, 2021
@@ -0,0 +1,10 @@
library("topGO")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that file should go and should probably be added to the .gitignore file

@bgruening
Copy link
Member

@combesf thanks for the adjustments. There are a few open questions.

@yvandenb yvandenb changed the title ProteoRE topGO enrichment analysis 20210412 [WIP] ProteoRE topGO enrichment analysis 20210412 Jun 2, 2021
@yvandenb yvandenb added the wip label Jun 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants