[video-codec] Roman Danyliw's Discuss on draft-ietf-netvc-testing-08: (with DISCUSS and COMMENT)
Roman Danyliw via Datatracker <noreply@ietf.org> Thu, 13 June 2019 12:50 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: video-codec@ietf.org
Delivered-To: video-codec@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id CAF1612004E; Thu, 13 Jun 2019 05:50:32 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Roman Danyliw via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-netvc-testing@ietf.org, Matthew Miller <linuxwolf+ietf@outer-planes.net>, netvc-chairs@ietf.org, linuxwolf+ietf@outer-planes.net, video-codec@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.97.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Roman Danyliw <rdd@cert.org>
Message-ID: <156043023275.12320.14106545312452480484.idtracker@ietfa.amsl.com>
Date: Thu, 13 Jun 2019 05:50:32 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/video-codec/DDFK6wmnYjidRHWNM84L7k1_Wzs>
Subject: [video-codec] Roman Danyliw's Discuss on draft-ietf-netvc-testing-08: (with DISCUSS and COMMENT)
X-BeenThere: video-codec@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Video codec BoF discussion list <video-codec.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/video-codec>, <mailto:video-codec-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/video-codec/>
List-Post: <mailto:video-codec@ietf.org>
List-Help: <mailto:video-codec-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/video-codec>, <mailto:video-codec-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 13 Jun 2019 12:50:33 -0000
Roman Danyliw has entered the following ballot position for draft-ietf-netvc-testing-08: Discuss When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-netvc-testing/ ---------------------------------------------------------------------- DISCUSS: ---------------------------------------------------------------------- (1) There appear to be deep and implicit dependencies in the document to the references [DAALA-GIT] and [TESTSEQUENCES]. I applaud the intent to provide tangible advice on testing and evaluation to the community with them. I have a few questions around their use. (1.a) Why aren’t [DAALA-GIT] and [TESTSEQUENCES] normative references as they are needed to fully understand the testing approach and provide the test data? (1.b) What should readers of the RFC do should these external references no longer be available? How is the change control of these references handled? (1.c) In the case of [DAALA-GIT] which version of the code in the repo should be used? Formally, what version of C is in that repo? (1.d) Per the observation that there are implicit assumptions made by the document about familiarity with [DAALA-GIT] and [TESTSEQUENCES], here are a few places where additional clarity is required: -- Section 4.3, Per “For individual feature changes in libaom or libvpx , the overlap BD-Rate method with quantizers 20, 32, 43, and 55 must be used”, what are libaom and libvpx and what is their role? -- Section 5.3. Multiple subsection in 5.3.* list what look like settings for tools (e.g., “av1: -codec=av1 -ivf -frame-parallel=0 …”). What exactly are those? How to read them/use them? (2) The full details of some of the testing regimes need to be more fully specified (or cited as normative): -- Section 3.1. The variable MAX is not explained in either equation. -- Section 3.1. This section doesn’t explain or provide a reference to calculate PSNR. I’m not sure how to calculate or implement it. -- Section 4.2. Reference needed for Bjontegaard rate difference to explain its computation -- The references [SSIM], [MSSIM], [CIEDE2000] and [VMAF] are needed to fully explain a given testing metric so they need to be normative (3) An IANA Considerations section isn’t present in the document. (4) A Security Considerations sections isn’t present in the document. ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- A few comments: (5) Consider qualifying the title to more accurately capture the substance of this draft “Video Codec Testing and Quality Measurement {using the Daala Tool Suite or Xiph Tools and Data}.” (6) Section 3.1 and 3.3. Cite a reference for the source code files names in question – dump_psnr.c and dump_pnsrhvs.c (which are somewhere in the [DAALA-GIT] repo?) (7) Editorial Nits -- Section 2.1. Expand PMF (Probability Mass Function) on first use. -- Section 2.1. Explain floor. -- Section 2.2. Typo. s/vidoes/videos/ -- Section 2.2. Typo. s/rewatched/re-watched/ -- Section 2.3. Typo. s/comparisions/comparisons/ -- Section 3.1. Expand PSNR (Peak signal to noise ratio) on first use. -- Section 3.1. Typo. s/drived/derived/
- [video-codec] Roman Danyliw's Discuss on draft-ie… Roman Danyliw via Datatracker