diStyApps / VisionCrafter

Craft your visions
130 stars 9 forks source link

Update main.py to add a general try catch on error, change slider ranges #9

Closed yavuzitconsulting closed 1 year ago

yavuzitconsulting commented 1 year ago

(commit message: add general try/catch to stop the application from window.close() on error, changed range slider for steps, changed range slider for video length)

it is impossible to troubleshoot this application in this phase when the application window (GUI) and console window closes whenever an exception occurs, i propose a general try/catch and log, so on fatal exceptions, it is at least possible to view the error message, even if the GUI crashes, this might be more helpful to development, but at this moment, i figure the community would like to report issues with their actual exception messages rather than not reporting them at all.

i also took the freedom to adjust the range of the steps and length slider, because i was able to create high-step and high-length videos without any issues, it just takes longer, maybe we can add a warning if the range exceeds n seconds.

thank you for your time, and for the tool, very happy to play around with this.

yavuzitconsulting commented 1 year ago

i noticed some errors that happened because vs-code auto-refactored the code, i will fix this and submit the request anew