Skip to content

Conversation

@Diniboy1123
Copy link
Contributor

Hey,

I wanted to insert metadata to VR optimized videos, but noticed that the Google official tool is Python2 only. So I decided to add support for the specific uuid box that the spherical video v1 spec requires to mp4ff to have a modern tool I can accomplish the same goal with.

I didn't implement v2 yet, because I didn't find a need for it.

Here is a test file from Google that parses just fine on my end with these changes now.

if err != nil {
return fmt.Errorf("piff senc: %w", err)
}
case "spherical-v1":
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would this be better off in its own function?

@tobbee
Copy link
Collaborator

tobbee commented Jan 7, 2026

@Diniboy1123 Thanks for the PR. Interesting format, but I'm a bit reluctant to add the XML parsing since it is not needed for the box parsing itself, and there is no inverse function to write a box from the parsed values, but just a simple encode of the raw XML: sw.WriteBytes([]byte(b.SphericalV1.XMLData)). Couldn't you just have an external program to parse the XML payload if needed and have the Info function just print the full XML?

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

Successfully merging this pull request may close these issues.

2 participants