-
Notifications
You must be signed in to change notification settings - Fork 212
New multi-course structure for training materials #486
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
Conversation
First step toward evolving Hello Nextflow to be more modular. Just content moves and stub creation (not updated content yet except some rough chopping where bits are getting split up).
✅ Deploy Preview for nextflow-training ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
work in progress
Much more step by step demonstration of how channels work + overall logical progression should make more sense now
Mainly to make it more clear what we're learning from sections 3 and 4
This one is really completely trivial now (no more nf-core style directory nesting). Remaining TODOs are just copy pasting the finalized code.
A few TODOs remaining, mainly updating console outputs etc
Text improvements Co-authored-by: Jonathan Manning <jonathan.manning@seqera.io>
Co-authored-by: Jonathan Manning <jonathan.manning@seqera.io>
A few updates left but this is almost complete
After reconsidering the review comments, realized that it is ok and actually better to introduce variable inputs and params in Hello World. Getting them out of the way before Channels is actually helpful. The trick is to not get bogged down in details about value versus queue channels. The rest is a bit streamlined as a results. Also made Operators intro more explicit. Also got started on some cleanup of the code files
- Space after view {...} - Config tweaks - demo to test profile - param json before profiles - Also renamed Run Nextflow to Nextflow Run
Mainly small text fixes Co-authored-by: Marcel Ribeiro-Dantas <mribeirodantas@seqera.io>
* No nav tabs * Don't hide the side nav on the homepages * Add instant navigation
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.
Part 2 is very nice! I requested some minor changes. Tomorrow I will review the next parts.
mostly small word changes Co-authored-by: Marcel Ribeiro-Dantas <mribeirodantas@seqera.io>
…extflow-io/training into gvda-streamline-hello-nextflow
small language tweaks Co-authored-by: Marcel Ribeiro-Dantas <mribeirodantas@seqera.io>
I have just minor comments (but not related to the content itself, as I believe it's just comments related to the content that was ported)
|
Major refresh of the training materials