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

FCPXML Import not linking to PNG files on import #659

Closed
tpayton opened this issue Sep 13, 2017 · 12 comments
Closed

FCPXML Import not linking to PNG files on import #659

tpayton opened this issue Sep 13, 2017 · 12 comments
Assignees
Labels
Milestone

Comments

@tpayton
Copy link

tpayton commented Sep 13, 2017

When exporting a scene for Final Cut Pro X, the resulting FCPXML file does not properly open in FCPX.

Repeatability: Always

STeps to repeat:

  1. Create a scene with several shots.
  2. From the file menu choose "Export Scene for Final Cut Pro and Premier"
  3. Reveal resulting FCPXML in the finder and drag onto FCPX.
    Results:
    Error message is shown.

screen shot 2017-09-13 at 11 08 41 am

Expected Results:
It works!

Analysis:
It appears that you are following the FCPXML specs exactly, according to this page (below) and yet it still doens't work. Frustrating. I'll do some futher research to see if I can discover what the issue is.

https://developer.apple.com/library/content/documentation/FinalCutProX/Reference/FinalCutProXXMLFormat/Resources/Resources.html

@audionerd
Copy link
Member

I noticed there are spaces in the filename "storyboarder test 1". If you make a new Storyboarder test without spaces in the filename, would it work?

@tpayton
Copy link
Author

tpayton commented Sep 13, 2017 via email

@audionerd
Copy link
Member

audionerd commented Sep 13, 2017

Fixed the bug. FCPX/Premiere export will now percentage-encode the urls representing the relative paths to the exported images.

Should be part of the Beta 12 release.

@audionerd audionerd added this to DONE in Features for RC1 Sep 13, 2017
@audionerd
Copy link
Member

I don't have FCPX so I'm not able to verify that this works. Could you try editing the "storyboarder test 1" .fcpxml file (the one with spaces) and editing the "src=" filenames to replace spaces with %20, then try importing? That would verify that this is indeed the fix we need...

Alternately, if you want to test this again when Beta 12 is released and report back that would be great 👍

@tpayton
Copy link
Author

tpayton commented Sep 14, 2017 via email

@audionerd
Copy link
Member

Could you attach your example XML directly to this GitHub issue?

#659

(GitHub apparently omits attachments sent via email)

@tpayton
Copy link
Author

tpayton commented Sep 14, 2017

Unfortunately encoding the urls with %20 still produces the import error in FCPX.

screen shot 2017-09-14 at 10 55 02 am

pastedgraphic-1

Puzzling because exporting a XML does encode it with %20 (see attached).

pastedgraphic-2

For the time being perhaps replacing spaces with an underscore in the filenames would be the most efficient way to give a quick fix.

@audionerd
Copy link
Member

In your example export from FCPX, how does the "src" line start? With a file://?

@tpayton
Copy link
Author

tpayton commented Sep 14, 2017

I tried a few combinations. No workie. In fact I was able to get FCPX to crash! I sent off a bug report to Apple on that one. I also asked the FCP team for any insight on the relative paths with encoded spaces. (I worked there for a short time, they should get back to me soon.)
screen shot 2017-09-14 at 11 16 38 am

@tpayton
Copy link
Author

tpayton commented Sep 14, 2017

BTW. Changing to underscores does work just fine in my tests. It might be the best solution for both Premiere and Final Cut.

@audionerd audionerd removed this from DONE in Features for RC1 Sep 15, 2017
@audionerd audionerd reopened this Sep 15, 2017
@audionerd audionerd added the bug label Sep 15, 2017
@audionerd audionerd added this to the Beta 13 milestone Sep 15, 2017
@audionerd audionerd added this to Beta 13 in Features for RC1 Sep 15, 2017
@audionerd audionerd self-assigned this Sep 15, 2017
@audionerd
Copy link
Member

Added an issue to track replacing spaces with dashes in exported filenames: #730

@audionerd audionerd moved this from Beta 13 to Beta 14 in Features for RC1 Oct 3, 2017
@audionerd audionerd modified the milestones: Beta 13, Beta 14 Oct 3, 2017
@audionerd audionerd moved this from Beta 14 to Beta 15 in Features for RC1 Oct 9, 2017
@setpixel setpixel moved this from Beta 15 to Beta 14 in Features for RC1 Oct 12, 2017
@audionerd audionerd moved this from Beta 14 to IN PROGRESS in Features for RC1 Oct 12, 2017
@audionerd audionerd moved this from IN PROGRESS to DONE in Features for RC1 Oct 13, 2017
@audionerd
Copy link
Member

We'll convert spaces to dashes in the upcoming release, which should fix this issue.

@audionerd audionerd removed this from DONE in Features for RC1 Oct 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants