twVolc / PyCamPermanent

Permanent PiCam (SO2) installation project software
GNU General Public License v3.0
1 stars 2 forks source link

Image warp error, throws error then stops processing #119

Closed tpering closed 1 month ago

tpering commented 3 months ago

image

tpering commented 3 months ago

Just a further note, this happened midway through processing, closing the software and reopening solved the problem and using different images, think it may have been due to the format of the Clear images used which were not the same as those taken by the camera (generated in Matlab) as these had different sizes

twVolc commented 2 months ago

Image registration in general does seem a bit buggy if jumping between different registration types then loading in a new sequence then checking the vignette-corrected image - I'm not sure the exact order of events but I've moved around these options a few times and found that OpenCV registration can sometimes revert back to no registration and vice versa.

twVolc commented 1 month ago

@tpering are you happy for me to close this issue, as it sounds like it was less a bug with the software and more an issue with the format of the data?

I need to open a new issue on general bugginess of image warping, particularly when loading in a new set of images, but think it would make sense to have that as a new issue.

tpering commented 1 month ago

Yes, please go ahead and close.

On Tue, Jun 4, 2024 at 9:11 AM twVolc @.***> wrote:

@tpering https://github.com/tpering are you happy for me to close this issue, as it sounds like it was less a bug with the software and more an issue with the format of the data?

I need to open a new issue on general bugginess of image warping, particularly when loading in a new set of images, but think it would make sense to have that as a new issue.

— Reply to this email directly, view it on GitHub https://github.com/twVolc/PyCamPermanent/issues/119#issuecomment-2146885014, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB36N25EZZ3VVLTSU5MT73DZFVZCXAVCNFSM6AAAAABFBL3STCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBWHA4DKMBRGQ . You are receiving this because you were mentioned.Message ID: @.***>

twVolc commented 1 month ago

Closing and moving registration bug discussions to #134