Opened 10 years ago

Closed 10 years ago

#220 closed defect (fixed)

ALF variable is not correctly assign when STAR_CROSS_SHAPES_LUMA is defined (encoder issue)

Reported by: chiayang_tsai Owned by:
Priority: minor Milestone:
Component: HM Version: HM-4.0
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

In Line 4826, TEncAdaptiveLoopFilter.cpp
filtNo is assigned as a temporary filter shape selection but not the final decision. The final decision in this for loop is cAlfParam.realfiltNo.

In Line 4930, TEcnAdaptiveLoopFilter.cpp
filtNo is used but not correctly assigned.

filtNo should be assigned as cAlfParam.realfiltNo somewhere before Line 4930.

The fix is as attached.

Change History (5)

comment:1 Changed 10 years ago by chiayang_tsai

The patch is sent to software coordinators.

Last edited 10 years ago by chiayang_tsai (previous) (diff)

comment:2 Changed 10 years ago by chiayang_tsai

  • Resolution set to fixed
  • Status changed from new to closed

fixed in r1422

comment:3 Changed 10 years ago by davidf

  • Component set to HM

Updating component after adding WD (Text) tickets

comment:4 Changed 9 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:5 Changed 9 years ago by davidf

  • Cc jct-vc@… added
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

  • Chia-Yang Tsai(Reporter, Participant)
  • David Flynn(Subscriber, Participant)
  • Frank Bossen(Subscriber)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)