There are several expect() calls in mujina-miner/src/board/bitaxe.rs that will panic on hardware failures or unexpected state. A panic takes down the whole process (or at minimum the task), leaving the system in an unknown state with hardware potentially half-initialized. If there's a reasonable chance something can fail, we should handle it with a proper error path.
Examples include expect() calls in spawn_stats_monitor that assume peripherals and channels are present. These should propagate errors instead of panicking.
Found during review of #33.
There are several
expect()calls inmujina-miner/src/board/bitaxe.rsthat will panic on hardware failures or unexpected state. A panic takes down the whole process (or at minimum the task), leaving the system in an unknown state with hardware potentially half-initialized. If there's a reasonable chance something can fail, we should handle it with a proper error path.Examples include
expect()calls inspawn_stats_monitorthat assume peripherals and channels are present. These should propagate errors instead of panicking.Found during review of #33.