-
Notifications
You must be signed in to change notification settings - Fork 12
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
TPV dry-run tool #48
Comments
This is a great idea. It would be very useful. |
I agree with @cat-bro - this is a really good idea @mira-miracoli |
Great idea @mira-miracoli! Just a few random thoughts and suggestions that crossed my mind.
Only the resubmit test case uses it at the moment, but since it is using Galaxy's own IntegrationTestCase class, the main advantage would be that it'll spin up a fairly comprehensive Galaxy instance, if we provide the |
Whoops, missed this issue, see #52. It's missing some things let arbitrary specification of tool params or input sizes, you can basically only specify a tool ID and a username/email currently, but it's a start. |
Thank you @natefoo |
We had the idea to make a small CLI TPV tool, that takes your config files, a few input params like
and then mocks a Galaxy instance, like in the unit tests, calculates the destination and gives a easy-to-read output, which destination was chosen and why. eg:
Additionally an explanation of how the rank was calculated in this specific case, would be helpful to debug errors in the configuration and check if it works as intentioned.
EU will work on this, but all help and suggestions are appreciated :)
The text was updated successfully, but these errors were encountered: