At a glance code is clean but VesselVio should adhere to some style guidelines that remove ambiguity and make it easier for contributors to develop and read. It looks like you already have flake8 and some rules set up which is good. Try setting up an autolinter if possible.
Big one for me personally is that I follow a literate programming style and almost never let myself or contributors merge code containing single letter variables. Makes reading code years later actually possible.
At a glance code is clean but VesselVio should adhere to some style guidelines that remove ambiguity and make it easier for contributors to develop and read. It looks like you already have flake8 and some rules set up which is good. Try setting up an autolinter if possible.
Big one for me personally is that I follow a literate programming style and almost never let myself or contributors merge code containing single letter variables. Makes reading code years later actually possible.