sagemath/sage

Fold src/bin/sage-num-threads.py into sagelib Python code

Closed this issue · 5 comments

As a follow-up to #29355 (which separated build-time from run-time
determination of the number of threads to use), we get rid of
src/bin/sage-num-threads.py completely by implementing the
desired defaulting behavior for the number of threads used in
src/sage/parallel/ncpus.py and src/sage/doctest/control.py.

Depends on #33317

CC: @jhpalmieri @slel

Component: refactoring

Reviewer: Matthias Koeppe

Issue created by migration from https://trac.sagemath.org/ticket/30639

slel commented

Description changed:

--- 
+++ 
@@ -1,2 +1,6 @@
-As a follow-up to #29355 (which separated build-time from run-time determination of number of threads to use), we get rid of `src/bin/sage-num-threads.py` completely by implementing the desired defaulting behavior for the number of threads used in `src/sage/parallel/ncpus.py` and `src/sage/doctest/control.py.
+As a follow-up to #29355 (which separated build-time from run-time
+determination of the number of threads to use), we get rid of
+`src/bin/sage-num-threads.py` completely by implementing the
+desired defaulting behavior for the number of threads used in
+`src/sage/parallel/ncpus.py` and `src/sage/doctest/control.py`.
 
comment:2

Setting new milestone based on a cursory review of ticket status, priority, and last modification date.

comment:5

Closing in favor of #33317.

Dependencies: #33317

Reviewer: Matthias Koeppe