Hello Config: more use of -output-dir (option B)#842
Conversation
…from outputDir
Larger refactor that:
- Uses -output-dir CLI option exclusively for controlling output directory
- Removes outputDir="results/${params.batch}" from config entirely
- Updates all run examples to use -output-dir
- Provides cleaner separation: -output-dir for base path, output block for subdirectory structure
…from outputDir
Larger refactor that:
- Uses -output-dir CLI option exclusively for controlling output directory
- Introduces -output-dir first, then removes hardcoded paths (logical order)
- Removes outputDir="results/${params.batch}" from config entirely
- Updates all run examples throughout sections 2.2 and 2.3 to use -output-dir
- Provides cleaner separation: -output-dir for base path, output block for subdirectory structure
✅ Deploy Preview for nextflow-training ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Nextflow linting complete! ❌ 1 files had 1 errors 💡 Tip: Click filename locations to go directly to that code. View all 1 issues
View formatting changes
|
-output-dir-output-dir (option A)
-output-dir (option A)-output-dir (option B)
pinin4fjords
left a comment
There was a problem hiding this comment.
I like this more incremental way of using the concepts, but it does drag a little bit. Middle way? ewels#2
|
Just realised that I need to make comparable edits to "Nextflow Run" - working on that now. |
Done in fa7f1bb |
- Fix solution file: change `.process` to `.name` in output path declarations. The `.process` attribute doesn't exist and causes "No such variable: process" error. - Fix documentation: remove invalid `outputDir` statements from workflow file code blocks. `outputDir` is a config file setting, not valid as a top-level statement in workflow files. This caused "Statements cannot be mixed with script declarations" error. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
When moving ${params.batch} from outputDir to path declarations,
the config file's outputDir needs to be updated accordingly.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Update cowpy call from collectGreetings.out to collectGreetings.out.outfile - Correct line number reference from 25 to 31 - Align comment text with actual source code Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
vdauwera
left a comment
There was a problem hiding this comment.
Looks great! I didn't run it manually but the new flow makes sense to me and all the explanations seem clear.
9360c85 to
58ff0e5
Compare
|
Ok, made a bunch of updates to the solutions and command output etc. I'm fairly confident that stuff is updated now. Good to merge I think. |
Possible implementation of what I was suggesting in #839
Warning
I haven't actually run this updated version yet, so the terminal outputs etc. are just written by hand.
If we like this content I will do that before merge.
This PR is what I would say is the "complete" version, rewritten to follow the kind of flow that I'd expect and showing how to combine use of both
-oand dynamic paths.It is longer and a little more complex, so I'm putting it here as a draft to compare against a more lightweight change which will be in PR #843