-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Add Working Groups section #196
Conversation
✅ Deploy Preview for cwl-website ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
This looks good to me, thank you!
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.
Text looks good! Rendered the site locally, entered the URL manually as the navigation menu points to the production site.
Clicking on the link returns an error page,
Adding the .html in the browser fixed it. Not sure if that's needed for GitHub Pages, or if there the deployment will be slightly different.
Thanks!
{: .table .table-striped .cols-2 } | ||
|Name|Scope| | ||
|--- |--- | | ||
|[CWL4HPC](/working-groups/cwl4hpc) | Identify workflow patterns capable of modeling large-scale scientific applications and implement the related CWL enhancement proposals.| |
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.
Maybe the link needs the .html
suffix?
2c21978
to
17763f1
Compare
content/working-groups/CWL4HPC.md
Outdated
- Motivate it with two real use cases that would benefit from the proposed feature; | ||
- Agree on a first draft of the syntax and semantics of the proposed feature; | ||
- Implement it as an extension on `cwltool` (the CWL reference implementation) and at least another CWL-compliant workflow system; | ||
- Validate it on at least two real CWL workflows. |
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.
Is it for existing workflows? How real does it need to be? I think maybe this could be difficult, @GlassOfWhiskey if we are adding something new like loops, cycles, ordered steps, anything new, maybe these new features do not work with existing workflows, and maybe we don't have any public workflow to be re-written with CWL and this new feature.
Maybe "Validate it on at least two existing CWL workflows where the proposal is applied, or with new example workflows created for this" (and maybe add it to the standard battery of tests?).
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.
Yes I agree. Better specifying it. I also forgot to add the test suite to the implementation part. I'll rephrase it
This commit adds a Working Groups section to track CWL focus groups on specific topics. The CWL4HPC working group has been added as an example, hoping that more activities will come in the future.
b0610d9
to
ff5b9c6
Compare
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.
Just read the latest changes, and it looks ready to be merged IMHO! THanks @GlassOfWhiskey !
This commit adds a Working Groups section to track CWL focus groups on specific topics. The CWL4HPC working group has been added as an example, hoping that more activities will come in the future.