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

Jump to bottom

Clean segment linking explanations #541

Merged
merged 13 commits into from Jan 2, 2022
Merged

Clean segment linking explanations #541

merged 13 commits into from Jan 2, 2022

Conversation

robUx4
Copy link
Contributor

@robUx4 robUx4 commented Aug 25, 2021

  • soft linking is not a real segment linking, it's a way to seek into other segments, there's no virtual segment involved (single presentation)
  • SegmentFilename, PrevFilename and NextFilename are mostly for file recovery
  • SegmentFamily is a generic thing usable by segment linking and chapter codec seeking
  • mark the NextUID/PrevUID as invalid for the first/last linked segment
  • make at least one NextUID/PrevUID mandatory per link node
  • group/split rules that apply to hard linking, medium linking or both
  • +add an example of chapter codec seeking/linking

Fixes #490

@robUx4 robUx4 added clarifications spec_main labels Aug 25, 2021
hubblec4
hubblec4 reviewed Aug 25, 2021
View changes
chapter_codecs.md Outdated Show resolved Hide resolved
hubblec4
hubblec4 reviewed Aug 25, 2021
View changes
notes.md Show resolved Hide resolved
hubblec4
hubblec4 reviewed Aug 25, 2021
View changes
notes.md Outdated Show resolved Hide resolved
hubblec4
hubblec4 reviewed Aug 25, 2021
View changes
notes.md Show resolved Hide resolved
@robUx4 robUx4 force-pushed the clean-segment-linking branch from c9bd43e to 54ecb8b Compare Aug 26, 2021
robUx4 added 13 commits Nov 28, 2021
@robUx4
make mandatory rules for linked segments clearer
27e3f23 
The restriction on tracks and timescale only apply to hard linking (segment splitting).

The placement of linked segments is not a hard requirement, it's up to the player
or system to make sure the segments are accessible for linking.
@robUx4
separate top sections in the Markdown file
d24f8c0
@robUx4
put the consecutive timestamp rule of hard linking after the tracks/t…
ecca64e 
…imestampscale rule

And clarify it.
@robUx4
clean markdown formatting
5bdeb4a
@robUx4
remove the MUST for NextUID/PrevUID on each segment
c776bb5 
It contradicts with the rules explained after.
@robUx4
simplify the rules on PrevUID/NextUID with a mandatory value per node
024fd86
@robUx4
mark the PrevUID of the first segment and NextUID of the last segment…
02f68a5 
… invalid

Plus some format cleaning
@robUx4
add SegmentFamily to the main linking section
653c2a9 
It's usually a good idea to mark linked segments with a SegmentFamily.
It's not specific to ordered chapters or chapter codecs.
@robUx4
clean some segment linking formatting
e6219d4
@robUx4
document how to use SegmentFilename, PrevFilename and NextFilename
f51506d
@robUx4
move the soft linking in the chapter codec spec
7393cb8 
Soft Linking is not actually a segment linking, it doesn't represent a virtual
segment to play from. It's just a way for chapter codecs to jump to other segments.
@robUx4
clean some medium linking text
7e7881b
@robUx4
clarify the NextUID precedence
d9ca8be
@robUx4 robUx4 force-pushed the clean-segment-linking branch from 54ecb8b to d9ca8be Compare Nov 28, 2021
@robUx4 robUx4 merged commit d28daa5 into master Jan 2, 2022
4 checks passed
@robUx4 robUx4 deleted the clean-segment-linking branch Jan 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hubblec4 hubblec4

Assignees
No one assigned
Labels
clarifications spec_main
Projects
Matroska specifications
Awaiting triage
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Rewrite parts of segment linking
2 participants
@robUx4 @hubblec4