search

dylanhthomas / peat-media-converter

A windows app to prepare any video or animated gif to be analyzed by PEAT (Photosensitive Epilepsy Analysis Tool)
Creative Commons Zero v1.0 Universal
10 stars 0 forks source link

Fail on initial test #9

Closed jolynyc closed 1 week ago

jolynyc commented 2 weeks ago

I gave it a test, and got a fail. This is mediainfo on the converted video

General Complete name : D:!!!!templates!!!isoc\Core Pack\Logo\STINGS\converter_output\siumulcast_converted.avi Format : AVI Format/Info : Audio Video Interleave Format profile : OpenDML Format settings : BitmapInfoHeader File size : 2.04 GiB Duration : 7 s 40 ms Overall bit rate : 2 488 Mb/s Frame rate : 25.000 FPS Writing application : Lavf59.17.102

Video ID : 0 Format : YUV Codec ID : I420 Codec ID/Info : 8 bit Y plane followed by 8 bit 2x2 subsampled U and V planes. Duration : 7 s 40 ms Bit rate : 2 488 Mb/s Width : 3 840 pixels Height : 2 160 pixels Display aspect ratio : 16:9 Frame rate : 25.000 FPS Scan type : Progressive Compression mode : Lossless Bits/(Pixel*Frame) : 12.000 Stream size : 2.04 GiB (100%)

To which PEAT responded "Can't Convert video stream". Choking on the 4K frame size?

dylanhthomas commented 2 weeks ago

@jolynyc Any chance you can send me a link to download the video and do some local testing?

dylanhthomas commented 2 weeks ago

@jolynyc It's odd, I was able to convert using both "Basic Processing" and "Full Processing" without issue, but I've come across issues in the past that were caused by different under-the-hood Windows configurations.

Couple of questions:

jolynyc commented 2 weeks ago

1.0.4

image

I am on Windows 10, and yes I use exclamation marks to give folders priority in alphanumeric listing. Has never been an issue.

jolynyc commented 2 weeks ago

And to be clear, I can do the conversion fine with the result as shown in mediaInfo above, but Trace Center Photosensitive Epilepsy Analysis Tool (PEAT) Version 1.6 does not accept the result.

dylanhthomas commented 2 weeks ago

Ahh, the problem is with PEAT. I misunderstood.

Here are my results testing the converted files:

"Basic"- file did not work...it's over PEATs 2GB-ish file limit. image

"Full" - file worked (60mb down-sampled to 640px) image

dylanhthomas commented 2 weeks ago

It's not so much frame size as file size. The limit depends somewhat on PEAT and somewhat on the system you're using. In my experience 1GB and smaller files always work. Above 1GB is hit or miss.

Part of the "Full" processing is downsampling the files to 640px, and if the file is bigger than 1GB, splitting the file into less than 1GB chunks.

Let me know if the Full processed file works for you. If not, there is something else we need to figure out.

Thanks =)

jolynyc commented 1 week ago

Yes, it doesn't say anything about an input file size limit. I assumed it would be resized to make it fit. Was surprised when it came out unchnaged at 4K.

On Thu, Oct 10, 2024 at 7:44 PM Dylan Thomas @.***> wrote:

It's not so much frame size as file size. The limit depends somewhat on PEAT and somewhat on the system you're using. In my experience 1GB and smaller files always work. Above 1GB is hit or miss.

Part of the "Full" processing is downsampling the files to 640px, and if the file is bigger than 1GB, splitting the file into less than 1GB chunks.

Let me know if the Full processed file works for you. If not, there is something else we need to figure out.

Thanks =)

— Reply to this email directly, view it on GitHub https://github.com/dylanhthomas/peat-media-converter/issues/9#issuecomment-2406234963, or unsubscribe https://github.com/notifications/unsubscribe-auth/AORVWRXF62S57AXU6ZL3VGTZ24GMZAVCNFSM6AAAAABPXDHUAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBWGIZTIOJWGM . You are receiving this because you were mentioned.Message ID: @.***>

dylanhthomas commented 1 week ago

Yeah, that's the basic vs full. There has been some confusion about it, so I think I need to do a better job describing the purposes. I also have an enhancement ticket to check if the output is over 1GB and set an alert.

If there is anything else, please let me know =)

jolynyc commented 1 week ago

Since output is a function of input, maybe a check on the input size on basic, with a choice to auto fall over to full?

On Fri, Oct 11, 2024 at 9:55 AM Dylan Thomas @.***> wrote:

Yeah, that's the basic vs full. There has been some confusion about it, so I think I need to do a better job describing the purposes. I also have an enhancement ticket to check if the output is over 1GB and set an alert.

If there is anything else, please let me know =)

— Reply to this email directly, view it on GitHub https://github.com/dylanhthomas/peat-media-converter/issues/9#issuecomment-2407468746, or unsubscribe https://github.com/notifications/unsubscribe-auth/AORVWRQ2YBSTFW2UFA7XIR3Z27KDTAVCNFSM6AAAAABPXDHUAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBXGQ3DQNZUGY . You are receiving this because you were mentioned.Message ID: @.***>