Skip to content

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

Merged
merged 69 commits into from
Feb 18, 2025

Conversation

vdauwera
Copy link
Collaborator

@vdauwera vdauwera commented Dec 16, 2024

Major refresh of the training materials

  • Pared down and streamlined the Hello Nextflow course (v2)
  • Spun out the genomics material into a separate mini-course, Nextflow for Genomics
  • Added stubs/placeholders for Nextflow Run and Side Quests
  • Removed the experimental and/or unfinished content from the training environment
  • Updated and streamlined the training index

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).
Copy link

netlify bot commented Dec 16, 2024

Deploy Preview for nextflow-training ready!

Name Link
🔨 Latest commit c99ec6f
🔍 Latest deploy log https://app.netlify.com/sites/nextflow-training/deploys/67b4b3edfe5be80008069962
😎 Deploy Preview https://deploy-preview-486--nextflow-training.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@vdauwera vdauwera self-assigned this Jan 2, 2025
A few TODOs remaining, mainly updating console outputs etc
vdauwera and others added 7 commits January 15, 2025 08:53
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
vdauwera and others added 6 commits February 17, 2025 08:30
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
Copy link
Member

@mribeirodantas mribeirodantas left a 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.

@maxulysse
Copy link
Collaborator

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)

  • Why mixing multi lines comments (with the /*...*/ and single line comments //?
    • Nowadays when I comment I use the shortcut from VS code and even multilines are using single comments, much less confusing I believe
  • Why not using the tag directive for the processes when we use the flatten operator, so that we see a bit more than (1) (2) or (3)
  • Why not using a params.yml over a params.json as json are really not meant to be edited by human hands

@vdauwera vdauwera merged commit 684138d into master Feb 18, 2025
8 checks passed
@vdauwera vdauwera deleted the gvda-streamline-hello-nextflow branch February 18, 2025 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants