Debounce autoreload of non web servers during development

TODO.md

@@ -3,7 +3,6 @@
 ## Top
 
 - Before new release
-  -- Fix order of forms that source / destination fields are at the top again
   -- test job_utils
   -- exclude autoreload when tests are saved (Custom Filter in server command watched files)
   -- Test canceled task/job in test_workers.py
@@ -112,6 +111,7 @@
 
 ## Maybe
 
+- Switch from Daphne to Uvicorn (maybe it has faster restart times during development)
 - Switch from Celery to Huey
 - Upgrade postgres server to v15, but we have to migrate the data then as the database files are incompatible a newer version
   -- <https://hollo.me/devops/upgrade-postgresql-database-with-docker.html>

adit/core/management/base/server_command.py

@@ -12,6 +12,8 @@
 from django.core.management.base import BaseCommand
 from watchfiles import PythonFilter, watch
 
+from ...utils.debounce import debounce
+
 
 class ServerCommand(BaseCommand, ABC):
     """See Django's runserver.py command.
@@ -57,6 +59,8 @@ def run(self, **options):
         if options["autoreload"]:
             self.stdout.write(f"Autoreload enabled for {self.server_name}.")
 
+            # We debounce to give the Django webserver time to restart first
+            @debounce(wait_time=2)
             def inner_run():
                 if self._popen is not None:
                     self._popen.terminate()

adit/selective_transfer/utils/debounce.py → adit/core/utils/debounce.py

@@ -5,6 +5,7 @@
 def debounce(wait_time=1):
     """Decorator that will debounce a function.
 
+    #
     Function is called after wait_time in seconds.  If it is called multiple times,
     it will wait for the last call to be debounced and run only this one.
     """

adit/selective_transfer/consumers.py

@@ -15,13 +15,13 @@
 
 from adit.accounts.models import User
 from adit.core.models import DicomNode
+from adit.core.utils.debounce import debounce
 from adit.core.utils.dicom_dataset import QueryDataset, ResultDataset
 from adit.core.utils.dicom_operator import DicomOperator
 from adit.core.utils.job_utils import queue_pending_tasks
 
 from .forms import SelectiveTransferJobForm
 from .models import SelectiveTransferJob, SelectiveTransferTask
-from .utils.debounce import debounce
 from .views import SELECTIVE_TRANSFER_ADVANCED_OPTIONS_COLLAPSED
 
 logger = logging.getLogger(__name__)