-
Notifications
You must be signed in to change notification settings - Fork 17
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
mxf2raw: extend --rdd6
option description
#45
Conversation
Thanks, it's very useful to have this documented more thoroughly. A few thoughts:
|
I've updated the PR to
The RDD 6 XML schema defines the |
43f364d
to
565b93b
Compare
That all sounds very good, thanks for all the improvements. Just one outstanding item:
On this topic, the help text says:
Perhaps you can just add something to convey whether this RDD 6 metadata being static is just an assumption, or whether mxf2raw actually checks that it's static (for the frames that it reads)? |
Yes, it assumes it is static and doesn't check if it isn't.
I've changed it to say it assumes it is static and that it uses the first and subsequent frames to accumulate the description text. |
LGTM |
Fixes #44