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

Initial commit for i2nca #777

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

Conversation

cKNUSPeR
Copy link

i2nca is a preprocessing software for MSI data in imzML format. Tools for conversion, QC reports and multi-file operations are included.

I am having some issues with collecting a dataset collection of the composite file imzML, so probably not all tools will work right away.

@cKNUSPeR cKNUSPeR marked this pull request as ready for review October 14, 2024 16:03
Copy link
Member

Choose a reason for hiding this comment

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

Is this file really needed?

@@ -0,0 +1,99 @@
<tool id="i2nca_agnostic_qc" name="i2nca Agnostic QC Report " version="@TOOL_VERSION@+galaxy@GALAXY_VERSION@">

Copy link
Member

Choose a reason for hiding this comment

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

please use the Galaxy Language Server and reformat those tools, so that indentation matches etc ...

</requirements>

<command detect_errors="exit_code"><![CDATA[
cp "${infile.extra_files_path}/imzml" indata.imzML &&
Copy link
Member

Choose a reason for hiding this comment

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

please use single quotes everywhere

Comment on lines +84 to +97
<citation type="bibtex">
@article{10.1093/gigascience/giab049,
author = {Cordes, Jonas and Enzlein, Thomas and Marsching, Christian and Hinze, Marven and Engelhardt, Sandy and Hopf, Carsten and Wolf, Ivo},
title = "{M2aia—Interactive, fast, and memory-efficient analysis of 2D and 3D multi-modal mass spectrometry imaging data}",
journal = {GigaScience},
volume = {10},
number = {7},
year = {2021},
month = {07},
issn = {2047-217X},
doi = {10.1093/gigascience/giab049},
url = {https://doi.org/10.1093/gigascience/giab049}
}
</citation>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<citation type="bibtex">
@article{10.1093/gigascience/giab049,
author = {Cordes, Jonas and Enzlein, Thomas and Marsching, Christian and Hinze, Marven and Engelhardt, Sandy and Hopf, Carsten and Wolf, Ivo},
title = "{M2aia—Interactive, fast, and memory-efficient analysis of 2D and 3D multi-modal mass spectrometry imaging data}",
journal = {GigaScience},
volume = {10},
number = {7},
year = {2021},
month = {07},
issn = {2047-217X},
doi = {10.1093/gigascience/giab049},
url = {https://doi.org/10.1093/gigascience/giab049}
}
</citation>
<citation type="doi">10.1093/gigascience/giab049</citation>

Use the force Luke!

Copy link
Member

Choose a reason for hiding this comment

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

Why not putting the citations into the macro and not repeat yourself?

Comment on lines +13 to +15
<requirements>
<container type="docker">biocontainers/i2nca:0.3.12_cv1</container>
</requirements>
Copy link
Member

Choose a reason for hiding this comment

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

Why not moving this to a macro?

@@ -0,0 +1,109 @@
<tool id="i2nca_splitter" name="Split a imzML file into smaller files with i2nca" version="@TOOL_VERSION@+galaxy@GALAXY_VERSION@">
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 to all tools a "profile='23.0'" can also be macrofied ...

mv ./outdata_B.ibd "${os.path.join($outfile_b.files_path, 'outfile_B.ibd')}" | true &&

echo "imzML A:" > $outfile_a &&
ls -l "$outfile_a.files_path" >> $outfile_a &&
Copy link
Member

Choose a reason for hiding this comment

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

single quotes

<data format="imzml" name="outfile_a" label="ROI A imzML, Conversion result of ${on_string}: imzML"/>
<data format="imzml" name="outfile_b" label="ROI B imzML, Conversion result of ${on_string}: imzML"/>
</outputs>

Copy link
Member

Choose a reason for hiding this comment

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

tests are missing?


]]></help>
<citations>
<citation type="bibtex">
Copy link
Member

Choose a reason for hiding this comment

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

move to macro?

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

Successfully merging this pull request may close these issues.

2 participants