-
Notifications
You must be signed in to change notification settings - Fork 6
add py requirement and actually output #44
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
base: master
Are you sure you want to change the base?
add py requirement and actually output #44
Conversation
Thanks @bernt-matthias for this update. I wander. Did you get an error with the new version? |
One of my users reported an error with v 0.1.0
So in this PR only quoting the python tool directory remains. |
Btw. shall we setup IUC style github actions on this repo? |
The version in the toolshed seems to still miss the python2 requirement. |
It should be working with Python 3 now |
Maybe. But I have no idea how it works (I was in parental leave the whole year). I could maybe also move these wrappers to IUC |
One still needs the requirement, e.g. for cases where the system has python 2. Let me just add it here. Just understood why my user used the old version: he is using the GTN workflow which is probably using 0.1.0
The change should be pretty simple. I could create a PR, and you would need to add the TS API keys in the settings.
But moving to IUC is also a good idea but probably more work. Up to you. |
and fix parameter names
a5f349f
to
cb42d33
Compare
Finished from my side. Should I bump the tool version? |
No description provided.