Switch to use_stdin=True by default for LSFCluster. (#388)

dask · Mar 5, 2020 · 450aed4 · 450aed4
1 parent 7e91119
commit 450aed4
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/dask_jobqueue/jobqueue.yaml b/dask_jobqueue/jobqueue.yaml
@@ -140,7 +140,7 @@ jobqueue:
     job-extra: []
     log-directory: null
     lsf-units: null
-    use-stdin: null             # (bool) How jobs are launched, i.e. 'bsub jobscript.sh' or 'bsub < jobscript.sh'
+    use-stdin: True             # (bool) How jobs are launched, i.e. 'bsub jobscript.sh' or 'bsub < jobscript.sh'
 
   htcondor:
     name: dask-worker

diff --git a/dask_jobqueue/lsf.py b/dask_jobqueue/lsf.py
@@ -55,8 +55,6 @@ def __init__(
 
         if use_stdin is None:
             use_stdin = dask.config.get("jobqueue.%s.use-stdin" % self.config_name)
-        if use_stdin is None:
-            use_stdin = lsf_version() < "10"
         self.use_stdin = use_stdin
 
         header_lines = []

diff --git a/docs/source/changelog.rst b/docs/source/changelog.rst
@@ -14,6 +14,7 @@ Development version
   of threads per process is roughly the same. Old default was to use one
   process and only threads, i.e. ``proccesses=1``, ``threads_per_process=cores``.
 - fix bug (forgotten async def) in ``OARCluster._submit_job`` (:pr:`380`).
+- ``LSFCluster``: switch to ``use_stdin=True`` (:pr:`388`).
 
 0.7.0 / 2019-10-09
 ------------------