Skip to content

WIP Modernize RGI tool and data manager#20

Open
bernt-matthias wants to merge 3 commits intoarpcard:masterfrom
bernt-matthias:modernize
Open

WIP Modernize RGI tool and data manager#20
bernt-matthias wants to merge 3 commits intoarpcard:masterfrom
bernt-matthias:modernize

Conversation

@bernt-matthias
Copy link

@bernt-matthias bernt-matthias commented Mar 13, 2025

Tool was requested by one of my users - but needs a few improvements which I plan to contribute here. Ping @raphenya, @agmcarthur, @innovate-invent, @arpcard

Starting this PR as work in progress:

  • add IUC style github workflows (only requires to set TS_API_KEY and TTS_API_KEY)
    • this will properly deploy tool and datamanager (DM is not yet deployed)
  • restructure repo a bit

Will continue here:

  • datamanager should download versioned data
  • to should not make use of __tool_data_path__ but use the paths defined in the data table
  • more tests needed

Also:

  • add to planemo monitor (to get containers for the data manager - the tool should already have one, because it uses only a single bioconda requirement)

@github-actions
Copy link

Stale pull request message

@github-actions github-actions bot added the stale label Sep 24, 2025
@natefoo
Copy link

natefoo commented Sep 26, 2025

We've had a request to install this tool on usegalaxy.org, the modernization here would go a long way to making this possible. Any chance this could get a review and merge?

@natefoo
Copy link

natefoo commented Jan 6, 2026

@bernt-matthias the DM was already moved to this repo, looks like your changes revert some of the modernization that was already done.

@bernt-matthias
Copy link
Author

I see. Seems that I was out of my mind.

@bernt-matthias
Copy link
Author

@raphenya @agmcarthur how should we proceed here? I'm happy to put some more work into this, but I will need a bit of feedback.

@tcollins2011
Copy link

I've received more requests to get the RGI wrapper updated and on galaxy main again. Is there anything that I could do to help? There are quite a few people requesting this now.

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.

4 participants