search

alyssaxuu / screenity

The free and privacy-friendly screen recorder with no limits 🎥
https://screenity.io
GNU General Public License v3.0
11.72k stars 904 forks source link

Audio loop every 30sec #205

Closed EdouardMD closed 7 months ago

EdouardMD commented 8 months ago

Screenity software currently has a bug where only the audio portion of the recording continuously loops, specifically the first 30 seconds.

Screenity : V3.1 chrome : V120.0.6099.217

alyssaxuu commented 8 months ago

Not sure I understand the issue - are you saying when you record with Screenity, the audio you record via microphone / system loops over the first 30 seconds? That seems very unusual if so, I would definitely need to investigate further.

Or is this about the editing feature for adding / replacing audio, where you can add your own audio file to add to the recording?

EdouardMD commented 8 months ago

Hi, no, let's imagine I'm recording a 2-minute video. The first 30 seconds of the video (only the audio) will be repeated every 30 seconds. The index seems to be poorly constructed. Only VLC seems to be able to play it randomly.

alyssaxuu commented 8 months ago

That seems very odd. If you attempt recovering the recording or using the option below to download troubleshooting data, does the video play fine? It might be an issue with transferring the recording data from the service worker into the editor page, since it's sandboxed. In larger videos the data is sent in batches due to the sendMessage file size limit (which also does not support blobs, so must be turned into base64 first), what might be happening is they're received out of order, some chunk is missing, or some batches are repeated causing the issue. I'm currently working on a patch to improve on this system and avoid this error (although I haven't been able to reproduce it in any machine / Chrome version).

The alternative could be an issue with IndexedDB itself that I'm using to store all the video chunks in while recording, if the recovered video also has the same problem. I will need to investigate nonetheless, hopefully have a patch ready in the coming days.

Edit: If you're on version 3.1.0 or above, could you try recording with backups enabled and checking if the video that is backed up has all the data?

EdouardMD commented 8 months ago

Hi, here is the output of the troubleshooting section : https://files.edouard.ml/share/znP-BsRr password : alyssax content :

The backed up video is also damaged. I will of course be happy to provide you with any additional information if needed.

alyssaxuu commented 8 months ago

Thank you so much for sharing this. I have looked into the file with ffprobe and there is a jump at the 30 second mark, where the frame goes back to 0 and up again, and once more at the 60 second mark. This is the case even on the raw recording file, so it's not a processing issue.

It's either an issue with the MediaRecorder itself (similar jump-y behavior has been reported in the past), or there was simply some sort of repetition / order issue when saving or retrieving the chunks in the IndexedDB somehow. Although the fact it only happens with the audio is really weird.

I've just made some changes to the recording process to try to prevent that, although unless I'm able to reproduce it it's hard to know if it will fix your issue. Probably won't be able to have a release ready in a few days though I'm afraid.

myktra commented 8 months ago

Hi there! Running in to this exact same issue, at roughly the 30 second mark and again at 60.

macOS 14.1, Chrome 120.

MichaelPrecel commented 7 months ago

Hey! I'm experiencing an issue with this as well unfortunately OS X 10.15.7 / Chrome 120

alyssaxuu commented 7 months ago

Hey! I'm experiencing an issue with this as well unfortunately OS X 10.15.7 / Chrome 120

Is this in the latest 3.1.4 version of Screenity? I still haven't been able to pinpoint the issue, I might need to replace the Localbase library with a different one since I'm afraid it might be the culprit.

MichaelPrecel commented 7 months ago

Hey @alyssaxuu – yep, it's on version 3.1.4.

alyssaxuu commented 7 months ago

@MichaelPrecel @myktra @EdouardMD I made a new release which replaces the old Localbase library with localForage for working with IndexedDB, and also has some other improvements. Could you please check if the issue is still there? If you reinstall from the Chrome Store you should be able to get the latest version (v3.1.5).

alyssaxuu commented 7 months ago

I'm closing this since I have not had any further reports of this particular issue, but do reopen if it's still the case.

MichaelPrecel commented 7 months ago

Thanks, I reinstalled and used yesterday without issue!

Cheers,

Michael Precel

Design & Development

Phone 0422 769 123 <+61422769123>

Social @mikepre https://www.instagram.com/mikepre/

Website michaelprecel.com https://www.michaelprecel.com/

Address 699 Nicholson St, Carlton North 3054 https://maps.app.goo.gl/QymeKa2VhsNqsAmF6

Hours 9:30am–5:00pm, Monday–Thursday

Please note, I check my inbox twice a day in the morning and afternoon. For anything urgent, please feel free to send me a text.

Working on unceded Wurundjeri and Boonwurrung Land

On Thu, 15 Feb 2024 at 4:21 am, Alyssa X @.***> wrote:

I'm closing this since I have not had any further reports of this particular issue, but do reopen if it's still the case.

— Reply to this email directly, view it on GitHub https://github.com/alyssaxuu/screenity/issues/205#issuecomment-1944274516, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGTPPZCI3WET6MOEVO7VOTTYTTXCTAVCNFSM6AAAAABB3OGIR2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBUGI3TINJRGY . You are receiving this because you were mentioned.Message ID: @.***>

EdouardMD commented 7 months ago

Hey @alyssaxuu, The issue seems to be resolved, thank you !