jackburton79 / bescreencapture

BeScreenCapture - Screen Capture software for Haiku
BSD 3-Clause "New" or "Revised" License
7 stars 10 forks source link

Mpeg - mp4 Works but only until 2000 frames is ok now #50

Closed GatoAmarilloBicolor closed 1 year ago

GatoAmarilloBicolor commented 2 years ago

Hello there, i was trying the new features of bescreencapture, and it work better now but still have a similar limit than the last time i did report wth more than 2000-2200-+ frames, if it record more frames give an encoding error. around more than 6 minutes(?

jackburton79 commented 2 years ago

Hi, will try with more than 2000 frames. is it full screen ?

GatoAmarilloBicolor commented 2 years ago

Yes, it is a full screen record, i did'nt see that answer before sorry, but yes, i am trying to do some gameplays on haiku, but it cant record more than 2000 frames, similar to how it was before but a little more frames.

jackburton79 commented 2 years ago

I managed to record more than 2000 frames once or twice, but I definitely had some issues at least once. I know that it takes some time to record so many frames, but what happens if you select "GIF" or "BMP" as file format ? This seems to work every time, so I'm starting to think there could be an issue in the encoding part

GatoAmarilloBicolor commented 2 years ago

image image

Those screenshots are for a video with more than 3000 frames, at the end it could not create the clip, this is the last version on the repository, later i will try with gif

GatoAmarilloBicolor commented 2 years ago

image the GIF format the "exporting" process never end, it take more than an hour with no end at this point.

jackburton79 commented 1 year ago

I've debugged this a bit. It fails in MovieEncoder:_WriteFrame(), on the line fMediaFile->WriteFrames(). For some reason, when you record more than 3000 frames, this fails with B_NO_INIT. This smells like memory corruption

jackburton79 commented 1 year ago

It was caused by setting the field rate to a value the media kit does not like. I disabled setting it for now.

Should be fixed in master. Could you try, please ?

jackburton79 commented 1 year ago

Fixed (in a better way) in master, with 9aa52e288e89e63ad0aea580cc00551e14678c06