Skip to content

STRY0020017 - Update Galaxy tool wrapper for StarAMR to version 0.12.1#276

Merged
emarinier merged 12 commits intomasterfrom
update_staramr
Mar 16, 2026
Merged

STRY0020017 - Update Galaxy tool wrapper for StarAMR to version 0.12.1#276
emarinier merged 12 commits intomasterfrom
update_staramr

Conversation

@emarinier
Copy link
Copy Markdown
Member

Description

As a data analyst, I would like the Galaxy tool wrapper for StarAMR to be updated to use the latest version (0.12.1), so that I can work with the updated StarAMR in Galaxy.

Acceptance Criteria

  • Update Galaxy StarAMR wrapper to use StarAMR 0.12.1 (latest release)

@emarinier emarinier self-assigned this Mar 11, 2026
@emarinier emarinier mentioned this pull request Mar 11, 2026
@apetkau apetkau self-requested a review March 11, 2026 16:01
Copy link
Copy Markdown
Member

@apetkau apetkau left a comment

Choose a reason for hiding this comment

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

This looks amazing Eric. Thanks so much for your great work 😄

One request I do have, can you change the list of selectable mlst schemes that are in the wrapper to match the most recent list from the updated mlst software?

<option value="auto" selected="true">Automatic</option>
<option value="aactinomycetemcomitans">aactinomycetemcomitans</option>
<option value="abaumannii">abaumannii</option>
<option value="abaumannii_2">abaumannii_2</option>

You can view the list by running:

mlst --list

On previous updates, I took this list of schemes from the mlst software and transformed into the XML format needed in the tool wrapper and copy/pasted those entries in.

@emarinier
Copy link
Copy Markdown
Member Author

Updated MLST options in c682d3a

Copy link
Copy Markdown
Member

@apetkau apetkau left a comment

Choose a reason for hiding this comment

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

Thanks so much for making all these changes Eric. I've approved this for whenever tests succeed.

@emarinier
Copy link
Copy Markdown
Member Author

Tests are failing because http://www.genomicepidemiology.org/services/ is down and has been down for at least a few days now.

@emarinier emarinier merged commit 29675aa into master Mar 16, 2026
18 of 20 checks passed
@emarinier emarinier deleted the update_staramr branch March 16, 2026 13:40
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