Skip to content

New tool: vg paths#8168

Open
Maed0x wants to merge 2 commits into
galaxyproject:mainfrom
Maed0x:add_vg_paths
Open

New tool: vg paths#8168
Maed0x wants to merge 2 commits into
galaxyproject:mainfrom
Maed0x:add_vg_paths

Conversation

@Maed0x

@Maed0x Maed0x commented Jul 7, 2026

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.

To request a review once your PR is ready, comment "please review" on the PR. This will
automatically apply the ready-for-review label if the PR is not a draft, all review
threads are resolved, and all CI checks have passed.


This PR adds vg paths to the vg tool collection. It is part of the current ongoing update of vg #7623
This PR also updates vg giraffe so that both tools share the same variables within the macros file + enabling installation on Galaxy 26.0

The missing data types have already been added in #22935. The profile was reduced to 26.0 so the tools can be installed on the European Galaxy instance for a thesis.

Thank you!

@SaimMomin12 SaimMomin12 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.

Some more stringent tests might be helpful

Comment thread tools/vg/paths.xml
<param argument="--exclude-sample" type="text" optional="true" label="Exclude from sample" help="Exclude paths belonging to this sample. Note: Cannot exclude the same sample when selecting paths by sample name"/>
<conditional name="output_con">
<param name="type" type="select" label="Output Type">
<option value="">Select an output type ...</option>

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
<option value="">Select an output type ...</option>
<option value="">Select an output type</option>

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.

2 participants