pietrop / digital-paper-edit-electron

Work in progress - digital paper edit project - Electron, Cross Platform Desktop app - Mac, Windows, Linux
https://pietropassarelli.net/autoedit
Other
103 stars 17 forks source link

WIN 10 64 bit - App starts process but slow to presents GUI #44

Open motioncircus opened 4 years ago

motioncircus commented 4 years ago

Downloaded and installed autoEdit-3-1.2.7.exe

Ran the exe (with Admin Rights), which started a Digital Paper Edit app (32 bit) process in Task Manager

To Reproduce Steps to reproduce the behavior:

  1. On a 64 bit WIN 10 Pro N machine, download the latest Pre-Release version
  2. Run the EXE
  3. Check Task Manager to see process Digital Paper Edit app (32 bit)
  4. Notice there's no application presenting

Expected behavior The App GUI to appear

Desktop (please complete the following information):

motioncircus commented 4 years ago

After writing this post, I find the App is now open... Perhaps it's just a LOT slower than I expect. I will report back if this is an intermittent issue Cheers

pietrop commented 4 years ago

Thanks for flagging this @motioncircus ! And thanks for the update

Interesting, it should probl be the 64bit version of the app rather then the 32bit.

based on electron builder docs it should be xcode10.2 on Travis, while in the current setup I have xcode 11 But not sure if this would effect the 32bit vs 64 bit.

And/or based on same docs, could add --x64 flag to make the arch explicit, in the npm script npm run build:mwl:publish:always

Following up on my comment on a related issue https://github.com/pietrop/digital-paper-edit-electron/issues/43#issuecomment-629300616 in v 1.2.7 I removed the electron remote module for the most part, but I suspect that there’s also some re architecting to do, to speed up performance. the electron “backend” at the moment is connect to the react client via a global window variable which might be effective performance. And need to figure out a better way for the two to communicate, while maintaining a modular architecture

some other clues in the electron docs about performance as well.

But back to the windows issue, since I don’t have a windows machine, would you be able to try and package/build locally with the 64bit arc flag and see if that improve startup time?

It would be a matter of following these instruction in the README#setup

to get setup

And modifying this line in package.json

- "build:w": "cross-env NODE_ENV=production electron-builder -w",
+"build:w": "cross-env NODE_ENV=production electron-builder -w --x64"

Before running

npm run build:w
motioncircus commented 4 years ago

Hi Pietro,

thanks for your swift response.

I'm very hopeful of being able to use Autoedit again and hoping I won't suffer the same issues I had a while back (maybe a year or two), where I struggled to replace the proxy files I used in my Resolve Edit, with the full resolution source footage files.

It seemed a simple path replacement task but caused me more headache than I could handle and I gave up due to deadline demands.

I'm a novice script hacker but I understand you would like me to build from source with your proposed variations. However when I run 'nmp' in my windows command prompt, I get this error:

[image: Capture.JPG]

nmp, is a linux command is it not?

I do actually have the Debian GNU/Linux command app https://www.microsoft.com/en-us/p/debian/9msvkqc78pk6 on my machine, am I to understand you mean for me to build using that?

To my novice mind, building an app using Linux would build a Linux APP, not a WIN10 app. However I stand to be corrected

I'll give it a go but, would you suggest I'd get better results running a full separate Linux install?

I don't trust Windows to run Linux properly. I used to have a separate Debian workstation but it got jettisoned in a recent house move.

N

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Sun, May 17, 2020 at 11:27 AM Pietro notifications@github.com wrote:

Thanks for flagging this @motioncircus https://github.com/motioncircus ! And thanks for the update

Interesting, it should probl be the 64bit version of the app rather then the 32bit.

based on electron builder docs https://www.electron.build/multi-platform-build#travis-macos it should be Xcode xcode10.2 on Travis, while in the current setup I have xcode 11 https://github.com/pietrop/digital-paper-edit-electron/blob/master/.travis.yml But not sure if this would effect the 32bit vs 64 bit.

And/or based on same docs, could add --x64 flag to make the arch explicit, in the npm script npm run build:mwl:publish:always https://github.com/pietrop/digital-paper-edit-electron/blob/master/package.json#L19

Following up on my comment on a related issue #43 (comment) https://github.com/pietrop/digital-paper-edit-electron/issues/43#issuecomment-629300616 in v 1.2.7 I removed the electron remote module for the most part, but I suspect that there’s also some re architecting to do, to speed up performance. the electron “backend” at the moment is connect to the react client via a global window variable which might be effective performance. And need to figure out a better way for the two to communicate, while many aiming a modular architecture https://textav.gitbook.io/textav-event-2018/projects/autoedit-panel-for-adobe-cep-pietro

some other clues in the electron docs about performance https://www.electronjs.org/docs/tutorial/performance as well.

But back to the windows issue, since I don’t have a windows machine, would you be able to try and package/build locally with the 64bit arc flag and see if that improve startup time?

It would be a matter of following these instruction in the README https://github.com/pietrop/digital-paper-edit-electron/blob/master/README.md

to get setup

And modifying this line in [package.json] ( https://github.com/pietrop/digital-paper-edit-electron/blob/master/package.json#L14 )

  • "build:w": "cross-env NODE_ENV=production electron-builder -w", +"build:w": "cross-env NODE_ENV=production electron-builder -w --x64"

Before running

npm run build:w

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pietrop/digital-paper-edit-electron/issues/44#issuecomment-629728907, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCDOEZDESBFYRHGOZZ5C7TRR44PFANCNFSM4NDERSYQ .

pietrop commented 4 years ago

Ok no worries, there should be another release coming up, in the release section 1.2.8 Where I added the 64 bit flag. 🤞 ( as of now it’s still building, check back in a bit)

Re npm, never mind those instruction, is probably a bit more involved, you’d need to install node ( the programming language) and npm is the package manager (works cross platform) but we might be able to test these changes without going through all that trouble.

And yeah it be good to get some feedback when you get to reconnecting the media in DaVinci Resolve.

One thing tho, is that autoEdit (2 and 3) works best if you import the original source (eg the media from the cards, or the source media you want to work with in the edit, rather then proxies if that makes sense), and let it create the audio and video preview for the app and the STT engine from it (uses ffmpeg under the hood), so that it reads the metadata from the original source footage and uses that info to create the sequence so that it can reconnect to it. ( this is mostly the EDL option)

motioncircus commented 4 years ago

Thanks,

I've heard of Node and once even tried to get my head around React but failed. I'll hang out for the next version from your end as you suggest.

I'm wondering about your apparent suggestion that I could use the Autoedit previews in place of proxies.. I'd love to be able to do that but doesn't that involve uploading the hundreds of GIGS of my 4K camera files to AssemblyAI or similar?

I haven't found a proxy settings dialogue and, when I considered trying such an approach, couldn't even locate the previews on my system.

Any suggestions?

Nigel

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Sun, May 17, 2020 at 12:02 PM Pietro notifications@github.com wrote:

Ok no worries, there should be another release coming up, in the release section https://github.com/pietrop/digital-paper-edit-electron/releases 1.2.8 Where I added the 64 bit flag. 🤞

Re npm, never mind those instruction, is probably a bit more involved, you’d need to install node ( the programming language) and npm is the package manager (works cross platform) but we might able to test these changes without going through all that trouble.

And yeah it be good to get some feedback when you get to reconnecting the media in DaVinci Resolve.

One thing tho, is that autoEdit (2 and 3) works best if you import the original source (eg the media from the cards, or the source media you want to work with in the edit, rather then proxies if that makes sense), and let it create the audio and video preview from it, so that it reads the metadata from the original source footage and uses that info to create the sequence so that it can reconnect to it. ( this is mostly the EDL option)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pietrop/digital-paper-edit-electron/issues/44#issuecomment-629731379, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCDOE7LGW4SEKD2WEC5PZTRR5ARVANCNFSM4NDERSYQ .

motioncircus commented 4 years ago

I just noticed your update and downloaded it but Windows decided it was a Virus and 'removed the threat'.

Any suggestions?

N Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Sun, May 17, 2020 at 12:12 PM Nigel Haslam nigelhaslam@gmail.com wrote:

Thanks,

I've heard of Node and once even tried to get my head around React but failed. I'll hang out for the next version from your end as you suggest.

I'm wondering about your apparent suggestion that I could use the Autoedit previews in place of proxies.. I'd love to be able to do that but doesn't that involve uploading the hundreds of GIGS of my 4K camera files to AssemblyAI or similar?

I haven't found a proxy settings dialogue and, when I considered trying such an approach, couldn't even locate the previews on my system.

Any suggestions?

Nigel

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Sun, May 17, 2020 at 12:02 PM Pietro notifications@github.com wrote:

Ok no worries, there should be another release coming up, in the release section https://github.com/pietrop/digital-paper-edit-electron/releases 1.2.8 Where I added the 64 bit flag. 🤞

Re npm, never mind those instruction, is probably a bit more involved, you’d need to install node ( the programming language) and npm is the package manager (works cross platform) but we might able to test these changes without going through all that trouble.

And yeah it be good to get some feedback when you get to reconnecting the media in DaVinci Resolve.

One thing tho, is that autoEdit (2 and 3) works best if you import the original source (eg the media from the cards, or the source media you want to work with in the edit, rather then proxies if that makes sense), and let it create the audio and video preview from it, so that it reads the metadata from the original source footage and uses that info to create the sequence so that it can reconnect to it. ( this is mostly the EDL option)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pietrop/digital-paper-edit-electron/issues/44#issuecomment-629731379, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCDOE7LGW4SEKD2WEC5PZTRR5ARVANCNFSM4NDERSYQ .

motioncircus commented 4 years ago

Hey Pietro,

I managed to successfully downloadautoEdit-3-1.2.8.exe https://github.com/pietrop/digital-paper-edit-electron/releases/download/1.2.8/autoEdit-3-1.2.8.exe by disabling Real Time Virus Monitoring, however, on restarting my machine, Windows Defender have me this error:

Capture2

Which looks more serious than your normal Windows Defender fake panic.

Is it possible that this Feury.c!cl virus has crept into your system?

N

Nigel Haslam | Director

motioncircus commented 4 years ago

Ok,

I've managed to wrangle Defender into silence on the matter of trojans and fired up your latest version, (autoEdit-3-1.2.8.exe, )however, as you see below, Task manager still seems to think that it's running the 32 bit version of the app.

Capture3

Nigel Haslam | Director

pietrop commented 4 years ago

Not sure re virus warning, probl not supposed to happen. they are packaged, built and shipped to the release section via Travis CI, a third party, with virtual machine for OSX.

Building another release 1.2.9, where tried to tweak the settings to see if it makes it for arch 64. 🤞

I'm wondering about your apparent suggestion that I could use the Autoedit previews in place of proxies.. I'd love to be able to do that but doesn't that involve uploading the hundreds of GIGS of my 4K camera files to AssemblyAI or similar?

what autoEdit does under the hood, is that it creates an audio preview / version of your media, (that is very fast to do even on large files) and then only sends that to the speech to text engine, eg assemblyAI.

I haven't found a proxy settings dialogue and, when I considered trying such an approach, couldn't even locate the previews on my system.

It also creates a lower res mp4 video preview, To use within the autoEdit programme to show you the preview. If that takes to long and the transcript is ready you’d be able to interact with it via the audio preview while the video preview is being created.

These audio and video previews are generally used only by autoEdit. I could make them available to “export”/“save” externally to the programme if useful tho.

If you have proxies made via the editing software, as I guess in a 4K workflow, you might not edit directly on those rushes (correct?), I’d be curious to look at the steps of that workflow in more details to see what might be an optimal way to integrate it with autoEdit, once we figure out the 32 vs 64 bit issue.

motioncircus commented 4 years ago

Thanks again for your support,

I'm greatly relieved to hear that AutoEdit is smarter than I'd guessed and only uploads low resolution versions for transcription. It might be worthwhile to mention this for others as dumb as myself.

I will try my normal workflow, which is simply to import my full resolution files (mostly 4K camera files), then to use DaVinci Resolve to create efficient low res proxies.

Very excited at my first test edit which worked straight out of the box.

I'll grab your next release as soon as I see it.

Cheers

Nigel

Nigel Haslam | Director

pietrop commented 4 years ago

Also when you get a chance could you check if autoEdit 2 windows version was also in 32bit?

https://opennewslabs.github.io/autoEdit_2/

Releases page https://github.com/OpenNewsLabs/autoEdit_2/releases

motioncircus commented 4 years ago

Sure,

N

Nigel Haslam | Director

motioncircus commented 4 years ago

It will be tomorrow I'm afraid,

I'm in Australia a bit out of sync with you I guess

Nigel

Nigel Haslam | Director

motioncircus commented 4 years ago

Morning!

Some news from downunder!

I've downloaded your latest version autoEdit-3-1.2.9 and thought I should flag that the download is double the size of the last one, at 420MEG.

The program is very slow to start still but it runs

Sorry to report that it's still showing up in Task Manager as 32 bit:

Capture5

FYI I just fired up my old install of Autoedit2 - autoEdit2-1.0.15.exe

And the process shows up in Task Manager like this:

Capture4

I've just done some tests

Test 1 made a paper edit, using 1080P proxiy files created in Handbrake from my 4K source footage and successfully imported it into DaVinci Resolve, .

Test 2 made a paper edit using the original 4K source footage which failed to import, throwing this errror in Resolve:

Capture6

So it would appear that, either I'm doing something stupid (highly possible), or Autoedit doesn't work with 4K files but I can't understand why because I've looked at the EDL that it spits out and there's nothing in there to do with resolution.

Any suggestions?

Nigel

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Sun, May 17, 2020 at 6:08 PM Nigel Haslam nigelhaslam@gmail.com wrote:

It will be tomorrow I'm afraid,

I'm in Australia a bit out of sync with you I guess

Nigel

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Sun, May 17, 2020 at 6:06 PM Nigel Haslam nigelhaslam@gmail.com wrote:

Sure,

N

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Sun, May 17, 2020 at 6:04 PM Pietro notifications@github.com wrote:

Also when you get a chance could you check if autoEdit 2 windows version was also in 32bit?

https://opennewslabs.github.io/autoEdit_2/

Releases page https://github.com/OpenNewsLabs/autoEdit_2/releases

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pietrop/digital-paper-edit-electron/issues/44#issuecomment-629759341, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCDOE2PQCHZNWP5OLE7DI3RR6K77ANCNFSM4NDERSYQ .

pietrop commented 4 years ago

Thanks for looking into it @motioncircus !

download size

Download size is probably double because it is now trying to do 32bit and 64bit in the installer.

virus warning

Do you still get a virus warning with latest version of autoEdit?

4K tests

The tests you did was with autoEdit 2 or 3?

What would be different in a 4K file?

The other thing to try is make a short sequence in Resolve with a few 4K clips, and export an EDL from Resolve to compare. (You could copy paste the content of the file here, might be easier then sending an attachment via GitHub)

screenshots

Seems like you might be replying to the GitHub email notifications with the screenshots, and the images are not coming through.

If you visit https://github.com/pietrop/digital-paper-edit-electron/issues/44 you should be able to add them here directly.

motioncircus commented 4 years ago

Hi Pietro, I've edited my replies above by dropping in the images - they seem to work as links but not directly visible in the post. Perhaps editing is different than direct posting because I've checked this post and the images are visible directly in preview.

virus warning

No, I didn't get the warning with the latest download but I did have Win Defender's 'real time virus monitoring' turned off, which is how I got around the issue yesterday.

4K tests

Were done in autoEdit 3

The Resolve import error I'm getting (on an autoEdit created EDL), says the clips fail to link because the timecode extents do not match any clip in the media pool

Which makes me wonder if it's possible that autoEdit-3 creates and uses timecode in it's previews that are stored in a different way, or a different place, than the timecode in the 4K files?

Capture6

Resolve test EDL export and comparison

I will make a new set of tests, using 4K files directly in autoEdit-3 AND a simple timeline created in Resolve and export the EDLs for comparison.

Back soon Nigel

motioncircus commented 4 years ago

Here's a screengrab of my two test EDLs, side by side in Notepad++

Capture11

There are some minor differences, one of which rings a bell from the last time I tried to figure this problem out - the fact that Resolve timelines default start at 01:00:00:00

The other is that Resolve sees the start time of each source clip as 02:xx:xx:xx where autoEdit sees source start times as 00:xx:xx:xx

I wonder if my rendering out proxy files eliminates this 4K two hour offset?

I hope this helps.

Nigel

motioncircus commented 4 years ago

OK I've done some research and testing and it's definitely a timecode offset issue because if I hack the EDL output of autoEdit, to add the required offset (and it's not just hours but minutes too) then it works imported into Resolve.

It seems my camera, a Panasonic GH4, is a bit flakey with timecodes:

This is a quote from the manual.

And of course I've been recording in MP4.

The timecodes on my generated proxies start at 00:00:00:00, hence them working fine when the edl is exported from autoEdit.

I might make a test using my GH4 with MOV format, after I check why I was using MP4, it might have been.

If you can think of a better workaround I'd love to hear it. Nigel

pietrop commented 4 years ago

That’s great, thanks!

4K EDL

Seems like it’s also expecting the audio to be on a separate EDL track?

I had a similar problem with .mxf files. But those where easier to recognize, because of the file extension, and if there’s an MXF in one of the clips autoEdit creates 3 EDL Tracks, one for video and two for audio.

file metadata

If the time codes are not recorded then in theory it would threat it as zero in autoEdit, but you said you added an offset manually?

autoEdit (both 2 and 3) reads the file metadata to see if it needs to compute a time code offset when creating the sequence. I’d be curious to see what are the metadata it reads for that 4K mp4.

The easier way to do this, is If you go in autoEdit2, if you have imported a 4K file in there already (?), and in the transcript view, under export option you choose json. You can then share the content of that json file here; or if you are familiar with json, open that with notepad++ 📝 and only share values of the metadata attribute. If that makes sense?

Otherwise there’s also a way to do it in autoEdit3 but is a little more complicated to explain. But can do so if the autoEdit2 option is not feasible.

EDL format

These are some notes I made a while when working on the module to create the EDL programmatically about how the EDL format works https://autoedit.gitbook.io/documentation/appendix/edl-format

pietrop commented 4 years ago

But yeah maybe test with mov first and see if that solves the problem before going down the metadata time codes rabbit hole 🐇 🕳

motioncircus commented 4 years ago

OK,

Yes there was an offset which I edited, so perhaps the GH4 manual is wrong, or a firmware update changed the timecode to allow it to be generated on MP4s.

Here's the (full) json export ((both flavours of json), from one of my original (MP4) 4K files imported into autoEdit3

4k_SOURCE_jsonAutoEditTest.zip

FYI Version 2.1.9 takes a full 4.5 minutes to start after double clicking the exe file - longest program load I've had for twenty years or so and very suspicious.

pietrop commented 4 years ago

Sorry, unfortunately in autoEdit 3, the transcript json doesn't return the metadata, unlike autoEdit 2.

But here's how to do it in autoEdit 3

  1. open autoEdit 3
  2. go to the transcript view
  3. click at the same time cmd+ option+i (not 100% sure keyboard combination on windows - maybe windows key + alt + i) OR in the menu under view --> Toggle Developer Tools
  4. This will show some dev tools. at the top, choose console if it's not already selected.
  5. in the console, you should see a json object that can be expanded, and you should be able to see the metadata info - see example below

dev tools

motioncircus commented 4 years ago

file metadata If the time codes are not recorded then in theory it would threat it as zero in autoEdit, but you said you added an offset manually?

That's right.. despite what the manual says, you can see a 2 hour odd offset (in the right hand side of the Notepad++ screengrab above)

I've just tested an MOV but the problem remains see below:

Annotation 2020-05-18 130050

It would appear that autoEdit isn't able to read the offset and add it to it's EDL output.

As you noticed, there is something different in the audio track creation too. Even when I bring in an EDL of a correctly adjusted timecode version, I can't see the audio in resolve until I manually create a new timeline and then switch back to my imported one.

Thanks for your help. I'm really hoping to be able to use this and I feel so close

N

pietrop commented 4 years ago

and, yeah re long startup time, might have to rollback to an earlier version

pietrop commented 4 years ago

and yeah see previous comment https://github.com/pietrop/digital-paper-edit-electron/issues/44#issuecomment-629920475 if you could see what metadata are actually being read by autoEdit 3, it would help narrow it down 🤞

motioncircus commented 4 years ago

I'm trying to follow your awesome guide GIF but my console seems a little different.. can't quite locate the json object

pietrop commented 4 years ago

Ok you can share a screenshot of where you got so far

motioncircus commented 4 years ago

A loong slooow restart fixed the view

Annotation 2020-05-18 132148

motioncircus commented 4 years ago

Looks like no timecode offset was picked up even from my MOV clip.

If you don't have anyone else suffering from this issue, I could simply transcode all my camera clips on injestion to my system, so they all have default timecode starting at zero. It works at least.

motioncircus commented 4 years ago

I think there may actually be another setting in the camera, instead of an accumulative timecode run, a fresh zero start for each clip. I might try that out too.

motioncircus commented 4 years ago

Sadly, no.. I can reset the time code to zero manually but after the first clip, the second one starts with the cumulative time from the end of the one before.

motioncircus commented 4 years ago

So it seems that autoEdit3 is currently not picking up the timecode start offset from my source clips, but Resolve is. Would you class this as a bug?

pietrop commented 4 years ago

Yeah might be a bug with the module that reads the metadata.

The info about the metadata camera timecode offset is there somewhere in the 4K video file. Otherwise Resolve would not see it either. (Best guess)

Could you share a short clip ( can just be a blank example) recorded from the camera of a few seconds in 4K so that (tomorrow) I can look into testing the metadata module with it and see the full range of what value it gets back. Might be that the module is reading the metadata from a different spot compared to your camera 🤞

pietrop commented 4 years ago

For context this is an example output of the full range of the metadata from a video clip

https://github.com/pietrop/digital-paper-edit-electron/blob/master/src/ElectronWrapper/lib/av-metadata-reader/examples/sampleMetadata.json

Could be that in GH4 or in 4K the time code info or the streams are setup differently

motioncircus commented 4 years ago

Thanks heaps for your support again. here's a link to a short 4K camera clip

https://drive.google.com/file/d/1_7DCkL1XyObBU9VRV2CVUTWscCmfKxwy/view?usp=sharing

and here's what Resolve makes of it's metadata

Annotation 2020-05-18 140422

and here's the result of a command line ffmpeg command to show it's metatdata, which reveals the timecode

Annotation 2020-05-18 141449

pietrop commented 4 years ago

Here is what I get from the metadata reader module that uses ffprobe in autoEdit 3, when I get the full range of metadata associated with that file

{
  "streams": [
    {
      "index": 0,
      "codec_name": "h264",
      "codec_long_name": "H.264 / AVC / MPEG-4 AVC / MPEG-4 part 10",
      "profile": "High",
      "codec_type": "video",
      "codec_time_base": "1/50",
      "codec_tag_string": "avc1",
      "codec_tag": "0x31637661",
      "width": 3840,
      "height": 2160,
      "coded_width": 3840,
      "coded_height": 2160,
      "has_b_frames": 1,
      "sample_aspect_ratio": "1:1",
      "display_aspect_ratio": "16:9",
      "pix_fmt": "yuvj420p",
      "level": 51,
      "color_range": "pc",
      "color_space": "bt709",
      "color_transfer": "bt709",
      "color_primaries": "bt709",
      "chroma_location": "left",
      "timecode": "N/A",
      "refs": 1,
      "is_avc": "true",
      "nal_length_size": 4,
      "id": "N/A",
      "r_frame_rate": "25/1",
      "avg_frame_rate": "25/1",
      "time_base": "1/90000",
      "start_pts": 7200,
      "start_time": 0.08,
      "duration_ts": 259200,
      "duration": 2.88,
      "bit_rate": 96541027,
      "max_bit_rate": "N/A",
      "bits_per_raw_sample": 8,
      "nb_frames": 72,
      "nb_read_frames": "N/A",
      "nb_read_packets": "N/A",
      "tags": {
        "creation_time": "2018-10-17 10:42:53",
        "language": "und",
        "timecode": "02:42:37:10"
      },
      "disposition": {
        "default": 1,
        "dub": 0,
        "original": 0,
        "comment": 0,
        "lyrics": 0,
        "karaoke": 0,
        "forced": 0,
        "hearing_impaired": 0,
        "visual_impaired": 0,
        "clean_effects": 0,
        "attached_pic": 0
      }
    },
    {
      "index": 1,
      "codec_name": "pcm_s16be",
      "codec_long_name": "PCM signed 16-bit big-endian",
      "profile": "unknown",
      "codec_type": "audio",
      "codec_time_base": "1/48000",
      "codec_tag_string": "twos",
      "codec_tag": "0x736f7774",
      "sample_fmt": "s16",
      "sample_rate": 48000,
      "channels": 2,
      "channel_layout": "unknown",
      "bits_per_sample": 16,
      "id": "N/A",
      "r_frame_rate": "0/0",
      "avg_frame_rate": "0/0",
      "time_base": "1/48000",
      "start_pts": 0,
      "start_time": 0,
      "duration_ts": 138240,
      "duration": 2.88,
      "bit_rate": 1536000,
      "max_bit_rate": "N/A",
      "bits_per_raw_sample": "N/A",
      "nb_frames": 138240,
      "nb_read_frames": "N/A",
      "nb_read_packets": "N/A",
      "tags": {
        "creation_time": "2018-10-17 10:42:53",
        "language": "und",
        "timecode": "02:42:37:10"
      },
      "disposition": {
        "default": 1,
        "dub": 0,
        "original": 0,
        "comment": 0,
        "lyrics": 0,
        "karaoke": 0,
        "forced": 0,
        "hearing_impaired": 0,
        "visual_impaired": 0,
        "clean_effects": 0,
        "attached_pic": 0
      }
    },
    {
      "index": 2,
      "codec_name": "unknown",
      "codec_long_name": "unknown",
      "profile": "unknown",
      "codec_type": "data",
      "codec_tag_string": "tmcd",
      "codec_tag": "0x64636d74",
      "id": "N/A",
      "r_frame_rate": "0/0",
      "avg_frame_rate": "25/1",
      "time_base": "1/90000",
      "start_pts": 0,
      "start_time": 0,
      "duration_ts": 259200,
      "duration": 2.88,
      "bit_rate": 800,
      "max_bit_rate": "N/A",
      "bits_per_raw_sample": "N/A",
      "nb_frames": 72,
      "nb_read_frames": "N/A",
      "nb_read_packets": "N/A",
      "tags": {
        "creation_time": "2018-10-17 10:42:53",
        "language": "und",
        "timecode": "02:42:37:10"
      },
      "disposition": {
        "default": 1,
        "dub": 0,
        "original": 0,
        "comment": 0,
        "lyrics": 0,
        "karaoke": 0,
        "forced": 0,
        "hearing_impaired": 0,
        "visual_impaired": 0,
        "clean_effects": 0,
        "attached_pic": 0
      }
    }
  ],
  "format": {
    "filename": "/Users/passarellip/Downloads/P1660367.MP4",
    "nb_streams": 3,
    "nb_programs": 0,
    "format_name": "mov,mp4,m4a,3gp,3g2,mj2",
    "format_long_name": "QuickTime / MOV",
    "start_time": 0,
    "duration": 2.88,
    "size": 35570170,
    "bit_rate": 98806027,
    "probe_score": 100,
    "tags": {
      "major_brand": "mp42",
      "minor_version": "1",
      "compatible_brands": "mp42avc1",
      "creation_time": "2018-10-17 10:42:53"
    }
  },
  "chapters": []
}
pietrop commented 4 years ago

ok, so from my example, I was looking under format.tags.timecode to find the timecode but it seems like it can also be under one of the stream under .tags.timecode 🤷‍♂️

so I tweaked the logic to also look inside the streams if it doesn't find it in the format attribute and now I can get it 🥳

{
  filePathName: '/Users/passarellip/Downloads/P1660367.MP4',
  fileName: 'P1660367.MP4',
  date: '2018-10-17 10:42:53',
  reelName: 'NA',
  timecode: '02:42:37:10',
  r_frame_rate: '25/1',
  fps: 25,
  duration: 2.88,
  sampleRate: 48000
}
pietrop commented 4 years ago

added a new version 1.2.10 (still building, windows version should show up shortly)

motioncircus commented 4 years ago

Pietro.. you rock!

I will try it out as soon as I can download V1.2.10

Being a novice at computer science, I'm not sure what the advantages are to using the 64 bit version but I don't care. I can even handle the 5 minute start up for the incredible advantages offered by your amazing program.

Thanks again

Nigel

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Tue, May 19, 2020 at 1:48 AM Pietro notifications@github.com wrote:

added a new version 1.2.10 https://github.com/pietrop/digital-paper-edit-electron/releases/tag/1.2.10 (still building, windows version should show up shortly)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pietrop/digital-paper-edit-electron/issues/44#issuecomment-630271036, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCDOE3MJMZRQ2YXU7XVQ2DRSFKF7ANCNFSM4NDERSYQ .

motioncircus commented 4 years ago

It works!

Fantastic Pietro.

I'm so excited.

I look forward to seeing where you're taking this next.

Happy to test WIN 10 versions for you to chase that elusive 64 bit version.

Nigel

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Tue, May 19, 2020 at 8:46 AM Nigel Haslam nigelhaslam@gmail.com wrote:

Pietro.. you rock!

I will try it out as soon as I can download V1.2.10

Being a novice at computer science, I'm not sure what the advantages are to using the 64 bit version but I don't care. I can even handle the 5 minute start up for the incredible advantages offered by your amazing program.

Thanks again

Nigel

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Tue, May 19, 2020 at 1:48 AM Pietro notifications@github.com wrote:

added a new version 1.2.10 https://github.com/pietrop/digital-paper-edit-electron/releases/tag/1.2.10 (still building, windows version should show up shortly)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pietrop/digital-paper-edit-electron/issues/44#issuecomment-630271036, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCDOE3MJMZRQ2YXU7XVQ2DRSFKF7ANCNFSM4NDERSYQ .

pietrop commented 4 years ago

Awesome, thanks! Yeah the 32bit vs 64bit might take a bit longer to figure out. Low priority for now.

And yeah the slow startup time, it’s most likely unrelated. I think I know what it is, but still exploring what a good fix would look like.

One question tho, once it starts, after the 5 min delay, are there other performance glitch ones, eg UI freezing or similar? Or is it overall usable?

motioncircus commented 4 years ago

Now you ask,

I have had a couple of issues with dialogue boxes refusing to accept text. For Instance when creating a new paper edit (or maybe it was a new transcript, I can't be sure), The dialogue box would open but I wasn't able to type into it.

Going forward, I will make a note and let you know exactly where it happens.

Cheers Nigel

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Tue, May 19, 2020 at 9:40 AM Pietro notifications@github.com wrote:

Awesome, thanks! Yeah the 32bit vs 64bit might take a bit longer to figure out. Low priority for now.

And yeah the slow startup time, it’s most likely unrelated. I think I know what it is, but still exploring what a good fix would look like.

One question tho, once it starts, after the 5 min delay, are there other performance glitch ones, eg UI freezing or similar? Or is it overall usable?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pietrop/digital-paper-edit-electron/issues/44#issuecomment-630490621, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCDOEZKMYYKWKXIR2GNNMLRSHBPBANCNFSM4NDERSYQ .

pietrop commented 4 years ago

That’ll be great, thanks!

pietrop commented 4 years ago

👋 @motioncircus, building another release 1.2.11 (still building should be ready shortly) that could have a performance improvement in initial startup time. Let me know if you get a chance to try it out when it's ready.

motioncircus commented 4 years ago

Great,

I'll check it out when I see it.. might be over the weekend now. Have a good one. N

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Fri, May 22, 2020 at 12:38 PM Pietro notifications@github.com wrote:

👋 @motioncircus https://github.com/motioncircus, building another release 1.2.11 https://github.com/pietrop/digital-paper-edit-electron/releases (still building should be ready shortly) that could have a performance improvement in initial startup time. Let me know if you get a chance to try it out when it's ready.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pietrop/digital-paper-edit-electron/issues/44#issuecomment-632445650, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCDOE2OOD6WZJSGKWFYE73RSXQRNANCNFSM4NDERSYQ .

pietrop commented 4 years ago

Thanks! No rush

motioncircus commented 4 years ago

Morning!

I downloaded and tested the latest version, autoEdit-3-1.2.12

It's a couple of MEGs bigger, @ 421,153 KB and it's working fine but Task Manager still reports it as 32 bit and it took the same 5 minutes to start up.

Cheers

Nigel

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Fri, May 22, 2020 at 12:50 PM Nigel Haslam nigelhaslam@gmail.com wrote:

Great,

I'll check it out when I see it.. might be over the weekend now. Have a good one. N

Nigel Haslam | Director

Motion Circus Pty Ltd. Byron Bay t: + 61 2 8007 7338 |m: + 61 403 020 126 w: www.motioncircus.com http://www.motioncircus.com/ link to website http://motioncircus.com

On Fri, May 22, 2020 at 12:38 PM Pietro notifications@github.com wrote:

👋 @motioncircus https://github.com/motioncircus, building another release 1.2.11 https://github.com/pietrop/digital-paper-edit-electron/releases (still building should be ready shortly) that could have a performance improvement in initial startup time. Let me know if you get a chance to try it out when it's ready.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pietrop/digital-paper-edit-electron/issues/44#issuecomment-632445650, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCDOE2OOD6WZJSGKWFYE73RSXQRNANCNFSM4NDERSYQ .

pietrop commented 4 years ago

Ok, cool, thanks for testing!

digital-pet commented 4 years ago

Just a tip, Microsoft makes free virtual machine images available for developers running Mac OS and Linux to use to test their windows builds. It might help you test some of these builds on your own!

https://developer.microsoft.com/en-us/windows/downloads/virtual-machines/

sqr commented 4 years ago

Hello Pietro, I am also experimenting very long loading times on Windows 10. Tried 1.2.12, 1.2.9 and 1.2.4 and all took around 5 minutes to load. It seems like you mentioned that the 64 bit flag was not the issue (although in my task manager the app is still called Digital Paper Edit app (32 bit)).

Is there any way I can help you find the solution for this? Do you have any idea of what might be causing it?

Thanks