Idea #18326
closed
- Target version set to 2022-01-19 sprint
- Assigned To set to Peter Amstutz
- Status changed from New to In Progress
- Target version changed from 2022-01-19 sprint to 2022-02-02 sprint
- doc/install/crunch2-cloud/install-dispatch-cloud.html.textile.liquid: the indentation of the config snippet is missing a few spaces in the 'NVIDIA GPU Support' section
- doc/install/crunch2-lsf/install-dispatch.html.textile.liquid: there is a spurious line break in the sentence
Use %% to express a literal %. The %%J in the default will be changed
to %J, which is interpreted by @bsub@ itself.
I see you duplicated that section from the configuration reference, would it be better to just refer to the configuration reference? It seems suboptimal to have two different copies of this text, they are bound to drift apart.
Otherwise, LGTM thanks.
Ward Vandewege wrote:
- doc/install/crunch2-cloud/install-dispatch-cloud.html.textile.liquid: the indentation of the config snippet is missing a few spaces in the 'NVIDIA GPU Support' section
- doc/install/crunch2-lsf/install-dispatch.html.textile.liquid: there is a spurious line break in the sentence
Fixed.
I see you duplicated that section from the configuration reference, would it be better to just refer to the configuration reference? It seems suboptimal to have two different copies of this text, they are bound to drift apart.
If we got clever and had anchors to the proper section of the configuration reference, that could work, but otherwise you're dumped onto a long page and have to go find it, which also seems suboptimal.
Otherwise, LGTM thanks.
Merged.
- Status changed from In Progress to Resolved
Also available in: Atom
PDF