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

startWithSAP & subsegmentStartsWithSAP attributes are not populated and this is not spec compliant #364

Closed
KameshDevarakonda opened this issue Mar 29, 2018 · 10 comments · Fixed by #1346
Labels
flag: spec compliance An issue with compliance to DASH, HLS, or other specs status: archived Archived and locked; will not be updated type: enhancement New feature or request

Comments

@KameshDevarakonda
Copy link
Contributor

System info

Operating System: all
Shaka Packager Version: v2.0.2 (and previous versions)

Issue and steps to reproduce the problem

When ShakaPackager generates DASH presentation, it doesn't indicate startWithSAP & subsegmentStartsWithSAP attributes either on AdaptationSet or Representation elements.

This is not spec compliant. From ISO/IEC 23009-1:2014(E), section 8.3 (on-demand profile) :
Representation elements with a @subsegmentStartsWithSAP value absent, zero or
greater than 3 may be ignored.

and 8.4 (live profile):
Representation elements with a @startWithSAP value (either supplied directly or inherited
from the containing) absent, zero or greater than 3 may be ignored.

@vaage vaage added the type: enhancement New feature or request label Mar 29, 2018
@vaage vaage added this to the Backlog milestone Mar 29, 2018
@vaage
Copy link
Contributor

vaage commented Mar 29, 2018

@KameshDevarakonda Thank you for filing this issue. I have labelled it as "enhancement" as it will require new additions to the code base. I have put it under backlog as our current milestones are already full. @kqyang, could you look at scheduling this work?

@kqyang kqyang modified the milestones: Backlog, v2.2 Mar 29, 2018
@kqyang kqyang modified the milestones: v2.2, v2.3 Aug 1, 2018
@kqyang kqyang modified the milestones: v2.3, v2.4 Nov 20, 2018
@kqyang kqyang modified the milestones: v2.4, v2.5 Feb 19, 2019
@kqyang kqyang added the flag: spec compliance An issue with compliance to DASH, HLS, or other specs label Aug 20, 2020
@kqyang
Copy link
Contributor

kqyang commented Aug 20, 2020

@koln67 Are you seeing this issue in your spec compliance check?

@koln67
Copy link
Contributor

koln67 commented Aug 20, 2020

I'm not seeing this error with the conformance tool. Is there an example of input data and a packager command which generated this compliance error?

@kqyang
Copy link
Contributor

kqyang commented Aug 22, 2020

@koln67 Maybe the conformance tool does not support the validation of startWithSAP yet. Can you see if you can find relevant code in the tool?

@kqyang kqyang modified the milestones: v2.5, v2.6 Jun 11, 2021
@XingzhaoYun
Copy link
Contributor

For ec-3 and ac-4 track, these attributes are required and should be "1".

@cosmin
Copy link
Contributor

cosmin commented Dec 2, 2021

Any progress on this issue by chance? To summarize the various SAP types in case it's helpful:

SAP type 1 should be used if for every fragment the first frame in decode order is also the first frame in presentation order (such as for fragments that start with I-frames)

SAP type 2 should be used when the first frame in decode order is not the first frame in presentation order, but the leading frames can be decoded (such as RADL in HEVC)

SAP type 3 should be used for open GOP encodings where the leading B-frames cannot be decoded and must be discarded (such as RASL for HEVC)

@kqyang
Copy link
Contributor

kqyang commented Dec 7, 2021

We don't have resources to work on it ourselves. You are welcome to help us implement the support and send a PR to us.

@XingzhaoYun
Copy link
Contributor

We don't have resources to work on it ourselves. You are welcome to help us implement the support and send a PR to us.

Would it be possible to just contribute audio part?

@joeyparrish
Copy link
Member

I don't see why not.

@XingzhaoYun
Copy link
Contributor

Here is a solution only for audio #1055

@joeyparrish joeyparrish modified the milestones: v2.6, Backlog Jun 24, 2022
cosmin added a commit that referenced this issue Feb 23, 2024
Add startwithSAP/subsegmentstartswithSAP for aac, ac3, ec3 and ac4 audio tracks according to LIVE or VOD profile.

Replaces #1055
Partial solution for #364

---------

Co-authored-by: Xingzhao Yun <xyun@dolby.com>
Co-authored-by: Joey Parrish <joeyparrish@google.com>
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Apr 24, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 24, 2024
@cosmin cosmin removed this from the Backlog milestone May 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
flag: spec compliance An issue with compliance to DASH, HLS, or other specs status: archived Archived and locked; will not be updated type: enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants