Batch of smaller fixes and update to version 1.12.1 (#114)

* fix: fixed a bug with removing pids from internal list

* Add type annotations to userfacing outputs

* Allways run with threading to process can be interrupted with ctrl-c

* Update pixi toml file

* chore: Add changelog entry and update versions

* Update pixi action

* Update pixi lock file

* Black formatting

Author: melund

.github/workflows/docs.yaml

@@ -20,7 +20,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - uses: prefix-dev/setup-pixi@v0.7.0
+      - uses: prefix-dev/setup-pixi@v0.8.1
         with:
           environments: docs
           cache-write: ${{ github.event_name == 'push' && github.ref_name == 'master' }}
@@ -52,7 +52,7 @@ jobs:
       #   run: |
       #     git checkout $(git describe --tags `git rev-list --tags --max-count=1`);
 
-      - uses: prefix-dev/setup-pixi@v0.7.0
+      - uses: prefix-dev/setup-pixi@v0.8.1
         with:
           environments: docs
           cache-write: ${{ github.event_name == 'push' && github.ref_name == 'master' }}

.github/workflows/test.yml

@@ -27,7 +27,7 @@ jobs:
     steps:
     - uses: actions/checkout@v4
 
-    - uses: prefix-dev/setup-pixi@v0.7.0
+    - uses: prefix-dev/setup-pixi@v0.8.1
       with:
         environments: test
         cache-write: ${{ github.event_name == 'push' && github.ref_name == 'master' }}
@@ -49,7 +49,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - uses: prefix-dev/setup-pixi@v0.7.0
+      - uses: prefix-dev/setup-pixi@v0.8.1
         with:
           environments: test
           cache-write: ${{ github.event_name == 'push' && github.ref_name == 'master' }}

CHANGELOG.md

@@ -1,5 +1,16 @@
 # AnyPyTools Change Log
 
+## v1.12.1
+
+**Fixed:**
+* Fixed a crash when stopping simulations with ctrl-c. It should now shutdown more gracefully.
+* Fixed a bug making it was hard to stop simulations which were not running in parrallel. Now
+  simulations are always started on separate threads, which makes it possible to stop them with ctrl-c.
+
+**Added:**
+* Added a some python annotations to the code base, which should provide better type hints in IDEs.
+
+
 
 ## v1.12.0
 

anypytools/__init__.py

@@ -36,7 +36,7 @@
     "NORMAL_PRIORITY_CLASS",
 ]
 
-__version__ = "1.12.0"
+__version__ = "1.12.1"
 
 
 def print_versions():

anypytools/abcutils.py

@@ -83,7 +83,7 @@ def add(self, pid):
 
     def remove(self, pid):
         with _thread_lock:
-            self._pids.pop(pid, None)
+            self._pids.discard(pid)
 
     def stop_all(self):
         """Clean up and shut down any running processes."""
@@ -914,12 +914,13 @@ def _worker(self, task, task_queue):
                 task.logfile = ""
             task_queue.put(task)
 
-    def _schedule_processes(self, tasklist) -> Generator[_Task, None, None]:
+    def _schedule_processes(
+        self, tasklist: list[_Task]
+    ) -> Generator[_Task, None, None]:
         # Make a shallow copy of the task list,
         # so we don't mess with the callers list.
         tasklist = copy.copy(tasklist)
-        number_tasks = len(tasklist)
-        use_threading = number_tasks > 1 and self.num_processes > 1
+        use_threading = "ANPYTOOLS_DEBUG_NO_THREADING" not in os.environ
         task_queue: Queue = Queue()
         threads: List[Thread] = []
         # run while there is still threads, tasks or stuff in the queue

pixi.lock

@@ -6,12 +6,10 @@ channels = ["conda-forge", "anybody" ]
 platforms = ["win-64", "linux-64"]
 
 [tasks]
-test = {cmd = "pytest tests", depends-on = ["install"]}
 develop = "pip install -e ."
 install = "pip install ."
 
 clean-build = "rm -rf build dist"
-build = {cmd="python -m build . --sdist"}
 pipy-upload = {cmd="twine upload dist/*"}
 release = {cmd="", depends-on=["test", "clean-build", "build", "pipy-upload"]}
 
@@ -50,6 +48,9 @@ debugpy = ">=1.8.1,<1.9"
 [feature.test.target.win-64.dependencies]
 anybodycon = "*"
 
+[feature.test.tasks]
+test = {cmd = "pytest tests", depends-on = ["install"]}
+
 
 [feature.docs.dependencies]
 python = "3.11.*"