-
Notifications
You must be signed in to change notification settings - Fork 6
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
Wrap for shp2msh and msh2vtu #66
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you clean this up a bit? There is a msh2vtu.xml
in the tools dir and there is a ogstools directory in the ogs_tools dir ... :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ogstools is now in conda
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a macros file with version tokens like here
galaxy-tools/tools/omero/macros.xml
Line 3 in c78bd5e
<token name="@TOOL_VERSION@">5.18.0</token> |
In order to use them, just import in the tools:
<import>macros.xml</import> |
Can be used in the tool
tag and for the requirement.
@tStehling please start a new PR for the last commit. To finish this one, just respond / fix according to the comments of @rmassei . |
tools/ogstools/Converter/macros.xml
Outdated
@@ -0,0 +1,18 @@ | |||
<macros> | |||
<!-- for historic reasons the omero-py version is used as the version for all tools --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<!-- for historic reasons the omero-py version is used as the version for all tools --> |
tools/ogstools/Converter/macros.xml
Outdated
<macros> | ||
<!-- for historic reasons the omero-py version is used as the version for all tools --> | ||
<token name="@TOOL_VERSION@">0.4.0</token> | ||
<token name="@PROFILE@">21.05</token> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<token name="@PROFILE@">21.05</token> | |
<token name="@PROFILE@">23.0</token> |
long_description: | | ||
A suite of tools for ogs. | ||
categories: | ||
- Ecology |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to find a fitting category. I'm going to ask Lars.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix category
No description provided.