-
Notifications
You must be signed in to change notification settings - Fork 56
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
EDL drifting out of sync #26
Comments
Note, NTSC is actually |
I tried this, with the node-timecodes module in terminal > var timecodes = require('node-timecodes');
undefined
> timecodes.fromSeconds(1427.4);
'00:23:47:10'
> timecodes.fromSeconds(1427.4, {frameRate: 29.97});
'00:23:47:NaN' There might be an issue with the fromSeconds function. |
See issue 25 for example. |
Hypothesis is that it drifts because of PAL vs NTSC frame rate differece. in autoEdit, it's default to PAL. Possible test/solution is
|
Figured it out, if I add an optional frame rate to the EDL module, make selection, export EDL, and then connect as NTSC the sequence in premiere works with no drift. timecodes.fromSeconds(this.tapeIn, {frameRate: 29.97}) so next step to fix this are
Then test it out again. |
As raised by one of the users when exporting an EDL file (paraphrasing)
To reproduce this issue
Get a video file
NTSC
,30fps
NTSC
,30fps
in the sequence.If bug occur, expecting it to drift of and not be same selection.
Further test. with same EDL file.
PAL
25fps
From previous tests expecting the text selection in the app to match video segments in the EDL sequence using when choosing PAL and 25fps
Considerations
The timecode module used in the EDL composer has a default setting on 25fps.
Possible fix
Allow to pass in fps to timecode as a variable and give the user the option to export the EDL as a NTSC sequence?
Need to look more into how timecodes and PAL/NTSC work.
The text was updated successfully, but these errors were encountered: