Close ni tasks

[AdvancedImagingUTSW]

Tried to create a more uniform way for closing tasks so that our console isn't cluttered upon exit.

There remains one traceback from the threading library.

If we replace raise SystemExit() with os._exit(1) in the following code this is eliminated completely:

        if event == "exception":
            if os.getenv("GITHUB_ACTIONS") == "true":
                return

            # Silence traceback to avoid printing to console.
            sys.tracebacklimit = 0
            raise SystemExit()
        return self.localtrace

I was hesitant to make this change because this method apparently does not invoke the normal Python cleanup routines (like flushing I/O buffers, calling finally blocks, etc.).

[codecov]

Codecov Report

Attention: Patch coverage is 45.92593% with 73 lines in your changes missing coverage. Please review.

Project coverage is 54.37%. Comparing base (f9b3532) to head (a8b9d74).
Report is 12 commits behind head on develop.

Files with missing lines	Patch %	Lines
src/navigate/model/devices/daq/ni.py	0.00%	16 Missing ⚠️
src/navigate/model/microscope.py	16.66%	10 Missing ⚠️
src/navigate/model/devices/filter_wheel/ni.py	0.00%	9 Missing ⚠️
src/navigate/model/devices/shutter/ni.py	14.28%	6 Missing ⚠️
.../model/devices/remote_focus/equipment_solutions.py	0.00%	5 Missing ⚠️
src/navigate/model/devices/lasers/ni.py	66.66%	4 Missing ⚠️
src/navigate/model/devices/zoom/dynamixel.py	0.00%	4 Missing ⚠️
src/navigate/controller/thread_pool.py	0.00%	3 Missing ⚠️
src/navigate/model/devices/filter_wheel/ludl.py	0.00%	3 Missing ⚠️
src/navigate/model/devices/mirrors/imop.py	0.00%	2 Missing ⚠️
... and 7 more

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1018      +/-   ##
===========================================
+ Coverage    54.32%   54.37%   +0.04%     
===========================================
  Files          180      180              
  Lines        20319    20395      +76     
===========================================
+ Hits         11038    11089      +51     
- Misses        9281     9306      +25     

Flag	Coverage Δ
unittests	54.37% <45.92%> (+0.04%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[annie-xd-wang]

Calling close_camera() here causes a DCAM error.