Opened 14 years ago Closed 14 years ago #42 closed defect (wontfix)parallel V2V (SymbolMode==3) broken in 0.7
Description
Dear Frank,
I noticed that the parallel V2V coding (symbolMode : 3) is broken in revision 0.6/0.7. This could be reproduced by setting symbolMode : 3 in the \cfg\encoder*.cfg files, and encoding. By comparing the encoded *.bin bitstream with that generated by setting "symbolMode : 1" (CABAC), it was noticed that the V2V bitstream was way bigger than that of the CABAC bitstream, while if V2V worked the way it supposed to be, the two bitstreams should be of the almost the same size.
A careful analysis showed that the default setting of
multiCodewordThreshold : 96000 # Threshold for multi-codeword coding
in the config file caused this problem. In the parallel V2V coding, only one CABAC is used to obtain the statistics for V2V coding, no matter what value "multiCodewordThreshold" is set.
The fix would be adding a validation check in TAppEncCfg.cpp:
Void TAppEncCfg::xCheckParameter()
…
After applying this fix, the size of the encoded bitstream using V2V (symbolMode :3) should be verified as very close to that of CABAC.
Besides, we did some improvement/cleaning for the V2V codes based on TMuC 0.7, and we attach the corresponding files
\source\App\TAppEncoder\TAppEncCfg.cpp
Please let us know how the fix/improvement goes, and if there is any problem, do not hesitate to contact us.
Best regards, Attachments (8)Change History (15)Changed 14 years ago by jzan@…Changed 14 years ago by anonymousChanged 14 years ago by anonymousChanged 14 years ago by anonymousChanged 14 years ago by anonymousChanged 14 years ago by anonymousChanged 14 years ago by anonymousChanged 14 years ago by anonymouscomment:1 Changed 14 years ago by anonymouscomment:2 Changed 14 years ago by fbossen
Is this simply a matter of telling people to use proper settings (i.e., set multiCodewordThreshold to 0 if symbolMode is 3)? comment:3 Changed 14 years ago by jzan@…
As TMuC 0.6/0.7 released for performance evaluation, people all around the world are using the default config setting for tool evaluation in TE12 and beyond.
Since not everyone has the knowledge of "proper settings", the default config files are considered as the suggested "proper setting" from knowledgeable experts. However, simply changing symbolMode == 3 in the default settings and running TMuC had produced incorrect results. This not only might lead to wrong conclusions, but also discourage future development on this topic. comment:4 Changed 14 years ago by fbossen
Parallel V2V not in HM comment:5 Changed 13 years ago by davidf
Updating component after adding WD (Text) tickets comment:6 Changed 13 years ago by davidf
comment:7 Changed 13 years ago by davidf
Note: See
TracTickets for help on using
tickets. | This list contains all users that will be notified about changes made to this ticket. These roles will be notified: Reporter, Owner, Subscriber, Participant
|
Hello, Frank,
I could mess up with some of the attachment files when creating the ticket for the first time, so I uploaded the files again which are indicated with 2 in the filenames.
Could you please try with those latter files first? Let me know if there is any issue.
Thanks.
Best regards,
Jinwen