VideoCapture dependency injection.

Author: ShlikhutkaBohdan

app_tools/my_video_capture.py

@@ -1,8 +1,10 @@
 import cv2
+import logging
 
 
 class MyVideoCapture:
     def __init__(self, video_source=0):
+        self.logger = logging.getLogger(f'{self.__class__.__name__}', )
         self.video_source = video_source
         self.vid = None
         self.width = None
@@ -13,7 +15,7 @@ def init(self):
         # Open the video source
         self.vid = cv2.VideoCapture(self.video_source)
         if not self.vid.isOpened():
-            print("Unable to open video source")
+            self.logger.error("Unable to open video source")
             raise ValueError("Unable to open video source", self.video_source)
 
         self.width = self.vid.get(cv2.CAP_PROP_FRAME_WIDTH)
@@ -26,22 +28,19 @@ def get_frame(self):
                 # Return a boolean success flag and the current frame converted to BGR
                 return (ret, cv2.cvtColor(frame, cv2.COLOR_BGR2RGB))
             else:
-                print("Ret not found")
+                self.logger.error("Ret not found")
                 return (ret, None)
         else:
-            print("get_frame Not opened")
+            self.logger.error("get_frame Not opened")
             return (None)
 
     # Release the video source when the object is destroyed
     def __del__(self):
-        print("Camera capture released")
+        self.logger.info("Camera capture released")
         if self.vid.isOpened():
             self.vid.release()
 
     def release_camera(self):
-        print("Camera capture released")
+        self.logger.info("Camera capture released")
         if self.vid.isOpened():
             self.vid.release()
-
-
-DEFAULT_VIDEO_CAPTURE = MyVideoCapture(0)

csc-manager.py

@@ -7,6 +7,7 @@
 
 from app_tools.logger import stdout_redirector
 from di.containers import MyContainer
+import ui
 
 
 def main():
@@ -21,5 +22,5 @@ def main():
     container = MyContainer()
     container.init_resources()
     container.config.from_ini('config.ini')
-    container.wire(modules=[sys.modules[__name__]])
+    container.wire(modules=[sys.modules[__name__]], packages=[ui])
     main()

di/containers.py

@@ -4,6 +4,8 @@
 
 from dependency_injector import containers, providers
 
+from app_tools.my_video_capture import MyVideoCapture
+
 
 class MyContainer(containers.DeclarativeContainer):
 
@@ -14,4 +16,6 @@ class MyContainer(containers.DeclarativeContainer):
         fname='logging.ini',
     )
 
+    default_video_capture = providers.Singleton(MyVideoCapture)
+
 

ui/camera_widget.py

@@ -1,15 +1,20 @@
 
 import PIL.Image
 import PIL.ImageTk
+from dependency_injector.wiring import Provide
 
+from app_tools.my_video_capture import MyVideoCapture
 from app_tools.qr_code_scaner import QrCodeScanner
-from app_tools.my_video_capture import MyVideoCapture, DEFAULT_VIDEO_CAPTURE
 import tkinter
 import imutils
 
 
 class CameraWidget:
-    def __init__(self, camera_frame, width, height, on_qr_scanned_callback=None, paused=False):
+    def __init__(self, camera_frame, width, height, on_qr_scanned_callback=None, paused=False,
+                 video_capture: MyVideoCapture = Provide['default_video_capture']
+                 ):
+
+        self.video_capture = video_capture
         self.camera_frame = camera_frame
         self.qr_code_scanner = QrCodeScanner()
         self.width = width
@@ -22,7 +27,7 @@ def __init__(self, camera_frame, width, height, on_qr_scanned_callback=None, pau
         self.canvas = tkinter.Canvas(self.camera_frame, width=self.width, height=self.height)
         self.canvas.pack()
 
-        self.vid = DEFAULT_VIDEO_CAPTURE
+        self.vid = self.video_capture
         self.delay = 40
 
         self.camera_frame.after(1000, self.update_barcode_frame)
@@ -47,12 +52,12 @@ def update_barcode_frame(self):
             self.camera_frame.after(self.delay * 100, self.update_barcode_frame)
 
     def pause(self):
-        DEFAULT_VIDEO_CAPTURE.release_camera()
+        self.video_capture.release_camera()
         self.paused = True
 
     def resume(self):
         if self.paused:
-            DEFAULT_VIDEO_CAPTURE.init()
+            self.video_capture.init()
             self.paused = False
             if self.camera_frame is not None:
                 self.camera_frame.after(100, self.update_barcode_frame)

ui/recovery_widget.py

@@ -4,13 +4,16 @@
 from controller.recovery_controller import RecoveryController
 from ui.recovery.header_widget import HeaderWidget
 from ui.recovery.footer_widget import FooterWidget
+import logging
 
 from tkinter import messagebox
 
 
 class RecoveryWidget:
     def __init__(self, recovery_frame):
         self.recovery_frame = recovery_frame
+        self.logger = logging.getLogger(f'{self.__class__.__name__}', )
+
         top_frame = tkinter.Frame(self.recovery_frame, borderwidth=3)
 
         currencies = coinFactory.get_available_currencies()
@@ -60,11 +63,11 @@ def show_coin_details_info(self, private_key, snip, address):
         self.footer_widget.show_coin_details_info(private_key, snip, address)
 
     def init_camera(self):
-        print("init recovery camera")
+        self.logger.info("init recovery camera")
         self.footer_widget.camera_widget.resume()
 
     def release_camera(self):
-        print("release recovery camera")
+        self.logger.info("release recovery camera")
         self.footer_widget.camera_widget.pause()
 
     @staticmethod