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

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

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

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

RSS Feeds Recent Issues | This issue
This is Rietveld