[PATCH] xvid zones patch doc (was [MPlayer-dev-eng] xvid zones patch)
Hi, (conversation move to the DOCS ML) On 6/1/05, Doom9 Feedback Hotline <feedback123@doom9.org> wrote:
Attached is the diff and the documentation for the patch.
Here's the patch for the man page. I think the examples should either go in the XviD section of the xml doc when I'll find the time to write it. What do you think? The last example is a little bit tricky, so if it's not available this might pollute the ML... Guillaume -- Jazz music and Open Source development looks like chaos to an untrained eye, but end-up producing masterpieces like "Kind of Blue" and "MPlayer". That's why I love them both... -- John Coltrane
Hi, Guillaume POIRIER wrote:
(conversation move to the DOCS ML) On 6/1/05, Doom9 Feedback Hotline <feedback123@doom9.org> wrote:
Attached is the diff and the documentation for the patch.
Here's the patch for the man page. I think the examples should either go in the XviD section of the xml doc when I'll find the time to write it.
Please leave at least one example in the man page. Trying to be _so_ reference like might be an easy temptation, but in the end it pisses of the people wanting to use the option. IMHO one of the Advantages of MPlayer man page is that it has examples in it. Diego, what is your opinion on the matter? Alex (beastd)
On Wed, 1 Jun 2005, Guillaume POIRIER wrote:
Hi, (conversation move to the DOCS ML) On 6/1/05, Doom9 Feedback Hotline <feedback123@doom9.org> wrote:
Attached is the diff and the documentation for the patch.
Here's the patch for the man page. I think the examples should either go in the XviD section of the xml doc when I'll find the time to write it.
+.I NOTE: +The quantizer option is not strictly enforced. +It affects only the planning stage of ratecontrol, and is still subject +to overflow compensation and qp_min/qp_max.
That's specific to x264/lavc. AFAIK, XviD does strictly enforce constant quant zones. --Loren Merritt
Hi, On 6/1/05, Loren Merritt <lorenm@u.washington.edu> wrote:
On Wed, 1 Jun 2005, Guillaume POIRIER wrote:
Hi, (conversation move to the DOCS ML) On 6/1/05, Doom9 Feedback Hotline <feedback123@doom9.org> wrote:
Attached is the diff and the documentation for the patch.
Here's the patch for the man page. I think the examples should either go in the XviD section of the xml doc when I'll find the time to write it.
+.I NOTE: +The quantizer option is not strictly enforced. +It affects only the planning stage of ratecontrol, and is still subject +to overflow compensation and qp_min/qp_max.
That's specific to x264/lavc. AFAIK, XviD does strictly enforce constant quant zones.
Woops. You're right. That's a bad side-effect of copy-and-paste from your paragraph. Fixed patch attached. Please check if the Nroff formatting is correct, I can't test it. Guillaume -- Jazz music and Open Source development looks like chaos to an untrained eye, but end-up producing masterpieces like "Kind of Blue" and "MPlayer". That's why I love them both... -- John Coltrane
On Wed, Jun 01, 2005 at 11:09:56PM +0200, Guillaume POIRIER wrote:
Fixed patch attached. Please check if the Nroff formatting is correct, I can't test it.
+.B zones=<zone0>[/\:<zone1>[/\:...]] +User specified quality for specific parts (ending, credits, ...) +(ABR or two pass). +Each zone is <start-frame>,<mode>,<value> where mode may be
Write the second mode as <mode>.
+.IPs "q" +constant quantizer override. +In that case, value=<2.0\-31.0> represents the quantizer value. +.IPs "w" +ratecontrol weigth override.
weight
+In that case, value=<0.01\-2.00> represents the quality correction in %.
I think there are better ways to to phrase both "In that case.." sentences, but my tired brain cannot come up with any right now ;) Feel free to commit, we can pick more nits later on.
+.RSs +.IPs zones=90000,q,20 +Encodes all frames starting with frame 90000 at constant quantizer 20. +.REss
The above line is superfluous and breaks the formatting. Diego
Hi, On 6/2/05, Diego Biurrun <diego@biurrun.de> wrote:
On Wed, Jun 01, 2005 at 11:09:56PM +0200, Guillaume POIRIER wrote:
+.RSs +.IPs zones=90000,q,20 +Encodes all frames starting with frame 90000 at constant quantizer 20. +.REss
The above line is superfluous and breaks the formatting.
Yeah, did not trust me too much on Nroff formatting. What you mean here is that I just have to delete the last ".REss", right? Guillaume -- Jazz music and Open Source development looks like chaos to an untrained eye, but end-up producing masterpieces like "Kind of Blue" and "MPlayer". That's why I love them both... -- John Coltrane
On Thu, Jun 02, 2005 at 11:48:36AM +0200, Guillaume POIRIER wrote:
Hi,
On 6/2/05, Diego Biurrun <diego@biurrun.de> wrote:
On Wed, Jun 01, 2005 at 11:09:56PM +0200, Guillaume POIRIER wrote:
+.RSs +.IPs zones=90000,q,20 +Encodes all frames starting with frame 90000 at constant quantizer 20. +.REss
The above line is superfluous and breaks the formatting.
Yeah, did not trust me too much on Nroff formatting. What you mean here is that I just have to delete the last ".REss", right?
Not the last one. You have to remove the line I labeled superfluous. Just look at it in man. Diego
Hi, On 6/2/05, Diego Biurrun <diego@biurrun.de> wrote:
On Thu, Jun 02, 2005 at 11:48:36AM +0200, Guillaume POIRIER wrote:
Hi,
On 6/2/05, Diego Biurrun <diego@biurrun.de> wrote:
On Wed, Jun 01, 2005 at 11:09:56PM +0200, Guillaume POIRIER wrote:
+.RSs +.IPs zones=90000,q,20 +Encodes all frames starting with frame 90000 at constant quantizer 20. +.REss
The above line is superfluous and breaks the formatting.
Yeah, did not trust me too much on Nroff formatting. What you mean here is that I just have to delete the last ".REss", right?
Not the last one. You have to remove the line I labeled superfluous. Just look at it in man.
Checked, fixed, and committed. Thanks. Guillame -- Jazz music and Open Source development looks like chaos to an untrained eye, but end-up producing masterpieces like "Kind of Blue" and "MPlayer". That's why I love them both... -- John Coltrane
Guillaume POIRIER wrote:
+.IPs "w" +ratecontrol weigth override.
"weight" I don't know if I"m happy with some of the rest, grammatically speaking (f'rinstance, having new sentences inside of a list which began in the middle of a sentence doesn't sit well with me), but I see no way to fix that without significant changes to the structure of the text, so since it's workable as it stands... -- The Wanderer Warning: Simply because I argue an issue does not mean I agree with any side of it. A government exists to serve its citizens, not to control them.
participants (5)
-
Alexander Strasser -
Diego Biurrun -
Guillaume POIRIER -
Loren Merritt -
The Wanderer