-
Notifications
You must be signed in to change notification settings - Fork 7
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
CWL Feature Extraction workflow #8
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.
Minor spelling errors.
CI workflow definition needs to be updated, if non-owners are supposed to submit PRs.
README.md
Outdated
|
||
#### 1. Install polus-plugins. | ||
|
||
- clone a image-tools reporsitory |
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.
- clone a image-tools reporsitory | |
- clone a image-tools repository |
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.
done
README.md
Outdated
- `pip install .` | ||
|
||
#### 2. Install workflow-inference-compiler. | ||
- clone a workflow-inference-compiler reporsitory |
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.
- clone a workflow-inference-compiler reporsitory | |
- clone a workflow-inference-compiler repository |
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.
fixed
When running the workflow as described in the README, I get an error, see stacktrace below: File "/Users/bengtljungquist/code/pr/image-workflows/workflow-inference-compiler/src/wic/api/pythonapi.py", line 250, in setattr |
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.
Please see comment in conversation regarding errors when running workflow, in addition to the spelling error.
@@ -0,0 +1,84 @@ | |||
# Common Workflow Language (CWL) Feature Extraction worflow |
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.
# Common Workflow Language (CWL) Feature Extraction worflow | |
# Common Workflow Language (CWL) Feature Extraction workflow |
@bengtl For security purposes, the CI has been configured to only run code from trusted users. Hamdah simply needs to change her github user to be a public member of the polusai organization. |
This PR extracts utilizes 8 distinct plugins and extract features