-
59221fd1 · Install required Python packages in the container
-
e5e2f1da · Add requirements for running the scripts
-
6066070e · Add find_brightest_sources step
-
964b2a19 · Add script to find brightest sources
-
d448282b · Merge branch 'rap-840-add-docs' into 'main'
- ... and 1 more commit. Compare e5ba73f2...d448282b
Perfect changes.
-
a8720d31 · Remove Docker installation note
- ... and 1 more commit. Compare 3bc47863...a8720d31
Then it's maybe better to remove the section here and add a section to running.rst
about running the workflow steps inside a container. In hindsigh...
Ah right! Good point. I added a comment to those sections that those directories should be mounted.
I think you should not assume that people will always specify an output directory inside the source directory. It will pollute the source directory...
It depends. I'm only saying that the text in your note is incorrect, as it depends on the options you pass to the CWL runner if separate workflow s...
Changed it, hope this sounds better?
Right, I thought I wouldn't mention the --no-container
parameter here yet. Do you think I should?
Yes, but presumably someone will mount a directory via -v <source_directory>:<mount_point>
(if they're using Docker), then the output directory, e....
main
, I'll ask the commissioners in the new year.