Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(5104)

Issue 1519: Update the coding tools draft. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 months ago by derf
Modified:
7 months, 3 weeks ago
Reviewers:
negge
Visibility:
Public.

Description

Describes an way to use a smaller multiply in the dyadic partition
 function, and removes the description of the non-dyadic partition
 functions, since we don't currently plan to use those, long-term.

Patch Set 1 #

Total comments: 8

Patch Set 2 : Update the coding tools draft. #

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
M doc/ietf/draft-terriberry-netvc-codingtools.xml View 1 5 chunks +84 lines, -93 lines 0 comments Download

Messages

Total messages: 5
derf
8 months ago #1
negge
r+, I understood these changes fine, but I also read your set of patches. http://review.xiph.org/1519/diff/2582/doc/ietf/draft-terriberry-netvc-codingtools.xml ...
7 months, 4 weeks ago #2
derf
7 months, 3 weeks ago #3
derf
https://git.xiph.org/?p=daala.git;a=commitdiff;h=ae21d4a4c9b9 http://review.xiph.org/1519/diff/2582/doc/ietf/draft-terriberry-netvc-codingtools.xml File doc/ietf/draft-terriberry-netvc-codingtools.xml (right): http://review.xiph.org/1519/diff/2582/doc/ietf/draft-terriberry-netvc-codingtools.xml#newcode421 doc/ietf/draft-terriberry-netvc-codingtools.xml:421: to a given CDF value can be determined ...
7 months, 3 weeks ago #4
negge
7 months, 3 weeks ago #5
r+

Thanks, this is much clearer.
Sign in to reply to this message.

RSS Feeds Recent Issues | This issue
This is Rietveld