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

Galaxy wrappers for the MFAssignR functions #567

Merged
merged 39 commits into from
Aug 15, 2024

Conversation

KristinaGomoryova
Copy link
Collaborator

This PR includes galaxy wrappers for all the MFAssignR functions except for the IsoFiltR function, which was solved in PR #565.

Closes #473
Closes #472
Closes #470
Closes #469

tools/mfassignr/macros.xml Show resolved Hide resolved
tools/mfassignr/mfassignr_histnoise.xml Outdated Show resolved Hide resolved
tools/mfassignr/mfassignr_histnoise.xml Outdated Show resolved Hide resolved
tools/mfassignr/mfassignr_histnoise.xml Outdated Show resolved Hide resolved
tools/mfassignr/mfassignr_histnoise.xml Outdated Show resolved Hide resolved
tools/mfassignr/mfassignr_mfassignCHO.xml Outdated Show resolved Hide resolved
tools/mfassignr/mfassignr_recal.xml Outdated Show resolved Hide resolved
tools/mfassignr/mfassignr_recallist.xml Outdated Show resolved Hide resolved
tools/mfassignr/mfassignr_snplot.xml Outdated Show resolved Hide resolved
tools/mfassignr/macros.xml Outdated Show resolved Hide resolved
@bgruening
Copy link
Collaborator

Very cool @KristinaGomoryova!

tools/mfassignr/mfassignr_isofiltr.xml Outdated Show resolved Hide resolved
tools/mfassignr/mfassignr_kmdnoise.xml Show resolved Hide resolved
help= "Input data frame - output from MFAssign or MFAssignCHO. Recommended are 'Unambiguous' assignments."/>
</inputs>
<outputs>
<data name="recal_series" format="tabular" label="Recalibration series by ${tool.name} on ${on_string}"/>
Copy link
Collaborator

Choose a reason for hiding this comment

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

think about adding column names to your tabular files - will improve UX

Copy link
Member

Choose a reason for hiding this comment

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

The tools are quite specific and most inputs are outputs from other tools, so I don't expect people to come with their own files. Or do you mean something else than adding the colnames to the help text?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

This creates some very tight coupling between the package and the Galaxy tools :/

@hechth hechth merged commit 87bb82e into RECETOX:master Aug 15, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants