<div dir="ltr">Sounds good to me.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 2, 2015 at 3:35 PM, Dylan Baker <span dir="ltr"><<a href="mailto:baker.dylan.c@gmail.com" target="_blank">baker.dylan.c@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I've gone ahead and deleted them.<br>
<div class="HOEnZb"><div class="h5"><br>
On Friday, January 02, 2015 01:36:45 PM Marek Olšák wrote:<br>
> Actually, if you want, you can remove these profiles. I don't use them<br>
> anymore and I don't know of anyone who does.<br>
><br>
> Marek<br>
><br>
> On Fri, Jan 2, 2015 at 3:15 AM, Dylan Baker <<a href="mailto:baker.dylan.c@gmail.com">baker.dylan.c@gmail.com</a>> wrote:<br>
> > You're right. Copy and paste error<br>
> ><br>
> > On Dec 31, 2014 6:15 PM, "Ilia Mirkin" <<a href="mailto:imirkin@alum.mit.edu">imirkin@alum.mit.edu</a>> wrote:<br>
> >><br>
> >> On Wed, Dec 31, 2014 at 8:46 PM, Dylan Baker <<a href="mailto:baker.dylan.c@gmail.com">baker.dylan.c@gmail.com</a>><br>
> >> wrote:<br>
> >> > The above commit changed r300.py and r500.py to no longer import<br>
> >> > quick.profile and modify that, but instead made them create a new<br>
> >> > profile with a single test. This is a major change in functionality,<br>
> >> > that appears to be unintentional.<br>
> >> ><br>
> >> > cc: <a href="mailto:laura@jlekstrand.net">laura@jlekstrand.net</a><br>
> >> > cc: <a href="mailto:marek.olsak@amd.com">marek.olsak@amd.com</a><br>
> >> > Signed-off-by: Dylan Baker <<a href="mailto:dylanx.c.baker@intel.com">dylanx.c.baker@intel.com</a>><br>
> >> > ---<br>
> >> ><br>
> >> > Alternatively, these profiles have been broken for some time, is it<br>
> >> > worth keeping them? Marek?<br>
> >> ><br>
> >> > tests/r300.py | 9 ++-------<br>
> >> > tests/r500.py | 13 ++++---------<br>
> >> > 2 files changed, 6 insertions(+), 16 deletions(-)<br>
> >> ><br>
> >> > diff --git a/tests/r300.py b/tests/r300.py<br>
> >> > index 7cb2a63..6731292 100644<br>
> >> > --- a/tests/r300.py<br>
> >> > +++ b/tests/r300.py<br>
> >> > @@ -2,18 +2,13 @@<br>
> >> > # Testing the r300 DRI driver<br>
> >> > #<br>
> >> ><br>
> >> > -from framework.profile import TestProfile<br>
> >> > -from framework.test import PiglitGLTest<br>
> >> > +from tests.quick import profile<br>
> >> ><br>
> >> > __all__ = ['profile']<br>
> >> ><br>
> >> > -profile = TestProfile()<br>
> >> > -<br>
> >> > -profile.tests['spec/!OpenGL 1.0/gl-1.0-blend-func'] = \<br>
> >> > - PiglitGLTest('gl-1.0-blend-func', run_concurrent=True)<br>
> >> > -env = profile.tests['spec/!OpenGL 1.0/gl-1.0-blend-func'].env<br>
> >> > # R300 blending hardware appears to be bad<br>
> >> > # Note that a setting of 1 bit is a special<br>
> >> > # case in Piglit that explicitly sets tolerance = 1.0f.<br>
> >> > +env = profile.tests['spec']['!OpenGL 1.0']['gl-1.0-blend-func'].env<br>
> >> > env['PIGLIT_BLEND_RGB_TOLERANCE'] = '2' # bits<br>
> >> > env['PIGLIT_BLEND_ALPHA_TOLERANCE'] = '2' # bits<br>
> >> > diff --git a/tests/r500.py b/tests/r500.py<br>
> >> > index 5476977..9f8ecc6 100644<br>
> >> > --- a/tests/r500.py<br>
> >> > +++ b/tests/r500.py<br>
> >> > @@ -2,18 +2,13 @@<br>
> >> > # Testing the r500 DRI driver<br>
> >> > #<br>
> >> ><br>
> >> > -from framework.profile import TestProfile<br>
> >> > -from framework.test import PiglitGLTest<br>
> >> > +from tests.quick import profile<br>
> >> ><br>
> >> > __all__ = ['profile']<br>
> >> ><br>
> >> > -profile = TestProfile()<br>
> >> > -<br>
> >> > -profile.tests['spec/!OpenGL 1.0/gl-1.0-blend-func'] = \<br>
> >> > - PiglitGLTest('gl-1.0-blend-func', run_concurrent=True)<br>
> >> > -env = profile.tests['spec/!OpenGL 1.0/gl-1.0-blend-func'].env<br>
> >> > # R500 blending hardware appears to be a bit better than R300<br>
> >> > # Note that a setting of 1 bit is a special<br>
> >> > # case in Piglit that explicitly sets tolerance = 1.0f.<br>
> >> > -env['PIGLIT_BLEND_RGB_TOLERANCE'] = '1' #bits<br>
> >> > -env['PIGLIT_BLEND_ALPHA_TOLERANCE'] = '1' #bits<br>
> >> > +env = profile.tests['spec']['!OpenGL 1.0']['gl-1.0-blend-func'].env<br>
> >> > +env['PIGLIT_BLEND_RGB_TOLERANCE'] = '2' # bits<br>
> >> > +env['PIGLIT_BLEND_ALPHA_TOLERANCE'] = '2' # bits<br>
> >><br>
> >> oops? (should be '1' based on the old values?)<br>
> ><br>
> ><br>
> > _______________________________________________<br>
> > Piglit mailing list<br>
> > <a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
> > <a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
> ><br>
</div></div><br>_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
<br></blockquote></div><br></div>