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

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 months, 1 week ago by derf
Modified:
3 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
4 months, 1 week 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 ...
4 months ago #2
derf
3 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 ...
3 months, 3 weeks ago #4
negge
3 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