Video Effect Crashes when Maintain aspect ratio is checked

[cfudala]

Describe the bug
Probably not the best thing to try, but I don't think it should crash. I wanted to see if it looked any better.
When selecting the Maintain Aspect Ratio check box on the Video effect, xLights crashes.
With the attached sequence, I did not get any dump files.
With my full sequence, after the second or third try I did get a dump file.
I have attached both files
To Reproduce
Import the sequence
go to Peace stakes
Select the video effect
check the box for Maintain Aspect ratio
Observe crash

Expected behavior
System should not crash.
Error or warning message should be displayed if this is not a appropriate thing to do.

Screenshots
none

Versions (please complete the following information):

• OS: Windows 11
• xLights version: 2025.03

Additional context
none

xLights_dbgrpt-23660-20250219T151628.zip

AspectRatioCheckBox.zip

Attachments

• If associated with a crash please attach/link to a dropbox containing the crash log that was produced
• If it crashed but no crash log was produced please use Tools Menu/Package Logs and attach/link to a dropbox containing the resulting zip file
• If the error relates to rendering an effect or a crash while sequencing please use Tools Menu/Package Sequence and attach/link to a dropbox containing the resulting zip file

If you would prefer not to post your sequences publically please say so and be ready to PM them to one of the developers when asked.

[cybercop23]

This issue has been fixed and will be available in the the next release. You can try the nightly build from github if you want to test it.

[cfudala]

Thanks, I'll check it out

…

On Wed, Feb 19, 2025 at 4:13 PM AlexB ***@***.***> wrote: This issue has been fixed and will be available in the the next release. You can try the nightly build from github if you want to test it. — Reply to this email directly, view it on GitHub <#5222 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AZBY2PHOQCYTZOY75TJTO7L2QT6YLAVCNFSM6AAAAABXPHJZ22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNRZHA3TQOBTGE> . You are receiving this because you authored the thread.Message ID: ***@***.***> [image: cybercop23]*cybercop23* left a comment (xLightsSequencer/xLights#5222) <#5222 (comment)> This issue has been fixed and will be available in the the next release. You can try the nightly build from github if you want to test it. — Reply to this email directly, view it on GitHub <#5222 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AZBY2PHOQCYTZOY75TJTO7L2QT6YLAVCNFSM6AAAAABXPHJZ22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNRZHA3TQOBTGE> . You are receiving this because you authored the thread.Message ID: ***@***.***>