Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flags do not override speed of recording. #19

Open
LX5321 opened this issue Mar 21, 2020 · 6 comments
Open

Flags do not override speed of recording. #19

LX5321 opened this issue Mar 21, 2020 · 6 comments

Comments

@LX5321
Copy link
Contributor

LX5321 commented Mar 21, 2020

Chronicler has been a great vscode extension for me but by specifying the speed as medium in the flags parameter makes a screen recording duration go from 1:00 minute to 0:45 minutes (i.e it's sped up instead of normal time recording). This is an issue when used with the record audio command. The audio is recorded for 1:00 minute but the video is of 0:45 minutes, thereby cutting the audio and ending the video.

@arciisine
Copy link
Owner

That is a fantastic point. Those two features weren't really built together, and have created a weird scenario. Are you expecting the audio to be sped up (sounding more like a chipmunk)? Or that the speed and the audio recordings shouldn't be supported together?

@LX5321
Copy link
Contributor Author

LX5321 commented Mar 21, 2020

What I think is happening here is

  • the audio is recorded via the underlying ffmpeg -f dshow which records audio from an input device.
  • FFmpeg triggers the command to record the vscode screen in default ultrafast preset.

@LX5321
Copy link
Contributor Author

LX5321 commented Mar 21, 2020

What I think can be done is allow audio recording only with preset as medium so that the video does not get over before the audio which is being recorded normally.

@LX5321
Copy link
Contributor Author

LX5321 commented Mar 21, 2020

What I am currently doing is, I am recording the video, then recording the audio, then overlaying it in sync with the code (I am explaining the code), when chronicler has the scope to do this all in one go!

@arciisine
Copy link
Owner

I'll take a look, and see what I can suss out. Sometimes I've run into platform specific issues, but hopefully this will be simple.

@LX5321
Copy link
Contributor Author

LX5321 commented Mar 21, 2020

That would be great! Thanks a bunch 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants