Opened 5 years ago
Closed 3 years ago
#918 closed defect (fixed)
StreamMergeApp outdated
Reported by: | ksuehring | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | VTM | Version: | VTM-8.0 |
Keywords: | Cc: | ksuehring, XiangLi, fbossen, jvet@… |
Description
The app for merging independently coded bistreams into a multiplexed bitstream using differend nuh_layer_id values is outdated.
The main problem is that layers can now share parameter sets, so a simple multiplex is likely to create collisions.
Additional issues:
- NAL unit header syntax is not updated (since an own implementation is used instead of library functions)
- VPS may not contain a proper description of the resulting bistream
- DCI is not filtered
- Start code emulations are not properly handled.
Should the App be removed? Is anybody contributing to fix it?
Change history (2)
comment:1 Changed 3 years ago by ksuehring
comment:2 Changed 3 years ago by ksuehring
- Resolution set to fixed
- Status changed from new to closed
MR1977 was not referenced here. It provided an improvement for the StreamMergeApp, although the functionality is still very limited. It seems OK to keep the app in this state, so I closed MR 2211.
https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/-/merge_requests/1977
Since there was no activity on maintaining/fixing StreamMergeApp over the last two years, the broken tool should be removed from the reference software.
An MR to remove StreamMergeApp was submitted: https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/-/merge_requests/2211