fix: toggle_auto_change_frame divides by zero with default fps

[andrewwhitecdw]

Problem

fix: toggle_auto_change_frame divides by zero with default fps

Fix

Replace:

    def set_auto_change_frame(self, new_bool):
        if (self.auto_change_frame == new_bool):
            return

        self.auto_change_frame = new_bool
        if (self.auto_change_frame):
            print("Start auto changing frame ...")
            wait_duration = 1.0 / self.auto_fps
            pyglet.clock.schedule_interval(self.visualize_next_frame, wait_duration)

with:

    def set_auto_change_frame(self, new_bool):
        if (self.auto_change_frame == new_bool):
            return

        if (new_bool and self.auto_fps <= 0):
            print("auto_fps is not set, cannot start auto changing frame")
            return

        self.auto_change_frame = new_bool
        if (self.auto_change_frame):
            print("Start auto changing frame ...")
            wait_duration = 1.0 / self.auto_fps
            pyglet.clock.schedule_interval(self.visualize_next_frame, wait_duration)

Files changed

• nvdu/viz/nvdu_viz_window.py

[andrewwhitecdw]

Closing this sweep-generated PR: sole commit is missing a valid Signed-off-by trailer. It does not meet the sweep requirements (single signed-off commit).