Use VideoCaptureMixin as capture_frame implementation

[trickeydan]

When the inheritance of classes is in any other order for Camera, it results in the abstract capture_frame implementation being used instead of the implementation in VideoCaptureMixin. This results in NotImplementedErrors whenever using either of these cameras. Interestingly, this is not caught by linting, typing or the runtime protections against executing abstractmethods.

Code to reproduce bug before this patch:

from zoloto.cameras import Camera
from zoloto.marker_type import MarkerType

class ZolotoCamera(Camera):

    marker_type = MarkerType.DICT_APRILTAG_36H11

    def get_marker_size(self, marker_id: int) -> int:
        return 80

cam = ZolotoCamera(0)

cam.capture_frame()

Possible inheritance orders:

• class Camera(BaseCamera, IterableCameraMixin, VideoCaptureMixin): - Nope
• class Camera(IterableCameraMixin, VideoCaptureMixin, BaseCamera): - Nope, idk why
• class Camera(VideoCaptureMixin, IterableCameraMixin, BaseCamera): - Yes

[RealOrangeOne]

Nice catch! Yeah mixin inheritance here is fun.

Surprised abc and mypy didn't catch this, but hey least it's a pretty simple fix.

Looks like there are a couple other cameras which use the "incorrect" inheritance order, which I'd be interested in also addressing in this PR.

[trickeydan]

Looks like there are a couple other cameras which use the "incorrect" inheritance order, which I'd be interested in also addressing in this PR.

Sorted.