search

lucianjames / Vid2Photogrammetry

Software to process videos into an image sequence optimised for use with photogrammetry software. A-level computer science project
1 stars 1 forks source link

Bugfixing before v1.1.0 release #24

Closed lucianjames closed 2 years ago

lucianjames commented 2 years ago

Will use this issue to record my bug finding/fixing that im going to perform soon before i do the v1.1.0 release.

lucianjames commented 2 years ago

image GUI scales weirdly. Quick fix in QT designer.

lucianjames commented 2 years ago

image Leaving frame count at a value of 0 causes the program to core dump

lucianjames commented 2 years ago

image Setting resize resolution too high (9999999.. in this case) causes the program to core dump

lucianjames commented 2 years ago

QOL: Add proper logging to the terminal, make it show whats going on in good detail. Currently it doesn't even tell me when its deleted an image due to blurriness!

lucianjames commented 2 years ago

image Not really a bug, but you can put do this with the output extension input lol. I need to make it a drop down list really, the way it currently works is a bit cringe.

lucianjames commented 2 years ago

GUI scales weirdly. Quick fix in QT designer.

image Fixed 2a6a697a7a0a6bc279f2ba8c53e7532aeb52cec7

lucianjames commented 2 years ago

Leaving frame count at a value of 0 causes the program to core dump

Fixed d18c8e84c3bd59bda2c80556248df255937dbb80

lucianjames commented 2 years ago

Setting resize resolution too high (9999999.. in this case) causes the program to core dump

Fixed 63345787a6645e9b92ad45ed440bfa19b27658c8 Added limit of 16384 to GUI (anything more than that is completely unreasonable, probably even many years into the future). This wont stop OOM core dumps from happening on weaker systems, but just get good lol.

lucianjames commented 2 years ago

QOL: Add proper logging to the terminal, make it show whats going on in good detail. Currently it doesn't even tell me when its deleted an image due to blurriness!

Fixed 093d090ba15653ae0729788e1ea91ddc57c45ba2 (plus a couple commits afterwards doing minor fixes)

lucianjames commented 2 years ago

Not really a bug, but you can put do this with the output extension input lol. I need to make it a drop down list really, the way it currently works is a bit cringe.

image

Added list in 8405c97324b9e7c820d1a5207ebce84e2b294216

lucianjames commented 2 years ago

Bugs fixed, ready for release v1.1.0