docs: Clarification of --cluster-stats docs & elaborating on the situation where job ids are not passed to the status script #1221
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR is a small clarification and elaboration on how to get --cluster-status to work .
The current docs were not clear to me about how to get this feature to work until I read them several times and had numerous false starts. So, I tried to spell out some of the information which was implied regarding how this feature works that I believe will make this more approachable to a wider audience.
Described a frequently faced challenge that must be solved for this feature to work as stated (well, must be solved by some number of users based on my hunting for answers prior to writing this PR. Specifically, enumerating a few workarounds for when the cluster submission script returns a string with more than the job id to snakemake, and snakemake then passes that entire string to the cluster-status script, causing it to best case crash the script, worst case the string contains characters which cause the shell to not even execute the script. The current docs, as written, imply snakemake has the job id and no thought need to be give to this aspect of the feature.
My intention was to clarify how this feature operated and make brief suggestions for solutions to the string, not job id, conundrum. Which, I do not consider a snakemake bug necessarily, it seems reasonable to ask users to parse this for themselves, they just should be aware it might need to be done.
QC
I added only a handful of sentences, which I proofread.
docs/
) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).