Skip to content

New tool : multiclassprediction #8041

Open
renu-pal wants to merge 17 commits into
galaxyproject:mainfrom
renu-pal:mcp
Open

New tool : multiclassprediction #8041
renu-pal wants to merge 17 commits into
galaxyproject:mainfrom
renu-pal:mcp

Conversation

@renu-pal

Copy link
Copy Markdown
Contributor

FOR CONTRIBUTOR:

  • I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • License permits unrestricted use (educational + commercial)
    [ *] This PR adds a new tool or tool collection
  • This PR updates an existing tool or tool collection
  • This PR does something else (explain below)

There are two labels that allow to ignore specific (false positive) tool linter errors:

  • skip-version-check: Use it if only a subset of the tools has been updated in a suite.
  • skip-url-check: Use it if github CI sees 403 errors, but the URLs work.

Comment thread tools/multiclasspredict/macros.xml Outdated
Comment on lines 11 to 19

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All requirements should have a version.

Comment thread tools/multiclasspredict/macros.xml Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This would be cool to have. Are you working on a conda recipe?

@renu-pal renu-pal May 31, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hi @bernt-matthias , yes I am working on a recipe as well, but its getting some errors and I don't know why its not working . Can you have a look there if you get some time.
bioconda/bioconda-recipes#65121
Thanks !

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

paths need to be single quoted

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would prefer if the actual source code would be published separately. Is there a citation?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use single quotes

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The method, parameters and outputs need detailed description. Without citation this is rather a blackbox.

@paulzierep paulzierep left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change to multi omics capability.
More help needed.

<xml name="requirements">
<requirements>
<requirement type="package" version="@TOOL_VERSION@">multi-omics-classification</requirement>
<requirement type="package" version="1.5.2">scikit-learn</requirement>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we can remove all this requirements, they are all in the multi-omics-classification recipe

@@ -0,0 +1,7 @@
scikit-learn

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this file can be removed, this is handled by conda

@@ -0,0 +1,10 @@
categories:
- Machine Learning
description: A FAIR Machine Learning Framework for Benchmarking, Feature Selection, and Hyperparameter Optimization of Machine Learning Models for Multiomics Data in Galaxy

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
description: A FAIR Machine Learning Framework for Benchmarking, Feature Selection, and Hyperparameter Optimization of Machine Learning Models for Multiomics Data in Galaxy
description: A Machine Learning Framework for Benchmarking, Feature Selection, and Hyperparameter Optimization of Machine Learning Models for Multiomics Data

description: A FAIR Machine Learning Framework for Benchmarking, Feature Selection, and Hyperparameter Optimization of Machine Learning Models for Multiomics Data in Galaxy
homepage_url: https://github.com/usegalaxy-eu/MultiOmicsClassification
long_description: |
The MultiClassPrediction tool is designed for multi-omics data analysis within the Galaxy platform, enabling reproducible and standardized model evaluation across diverse biological datasets.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

explain what it does, the galaxy part can be removed, we know its in galaxy :)


<inputs>

<param name="X" type="data" format="tabular" label="X: Input features" help="Feature matrix (independent variables used for prediction)"/>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

explain input format, first row is feature names ?

</inputs>

<outputs>
<data name="output" format="csv" label="Multiclass AUC results"/>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can we ouput tabular ?

<param name="sampling_strategy" value="No Sampling"/>
<param name="grid_search" value="false"/>

<output name="output">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please add real test output, also for the plots and compare the files

MultiClassPredict
==================

A FAIR Machine Learning Framework for Benchmarking, Feature Selection, and Hyperparameter Optimization of Machine Learning Models for Multiomics Data in Galaxy

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we do not need FAIR and Galaxy ref, we know its in Galaxy which is FAIR :)

@@ -0,0 +1,178 @@
<tool id="multi-omics-classification" name="multi-omics-classification" version="@TOOL_VERSION@+galaxy@VERSION_SUFFIX@" profile="@PROFILE@">
<description>Multi Class prediction </description>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
<description>Multi Class prediction </description>
<description>Multiomics Classification</description>


<inputs>

<param name="X" type="data" format="tabular" label="X: Input features" help="Feature matrix (independent variables used for prediction)"/>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actually, we could make this a real multi-omics tool by making this a repeat, similar to

<repeat name="omics" min="0" title="Multiple omics layers?">
                    <param name="train_omics" type="data" optional="true" format="tabular" label="Training omics data"/>
                    <param name="test_omics" type="data" optional="true" format="tabular" label="Test omics data"/>
                    <param name="assay" type="text" optional="true" label="What type of assay is your input?" help="This would be used as output name.">
                        <sanitizer invalid_char=" ">
                            <valid initial="string.letters">
                                <add value="_"/>
                            </valid>
                        </sanitizer>
                    </param>
                </repeat>

Then we can run the tool for each omics type using this as the label.
In the command line, one would need to merge X from all omics types once to create the combined input.

@paulzierep paulzierep left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can you give the test-data better names, like prediction_results.csv (tabular would be better), also for the plots

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Triage/Discuss

Development

Successfully merging this pull request may close these issues.

3 participants