<div dir="ltr">Ideally I'd figure out why our timeout mechanism + concurrency = double free's in python and we'd just use timeouts and not need any printout at all. Which was my plan I was sidetracked from...<div><br></div><div>Also, with the atomic backend those numbers should line up with the file numbers (they are not generated from the same source, but they are generated with the same pattern) in ${results_dir}/tests, which makes your statement "the number is meaningless" not completely true :)</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 24, 2014 at 12:12 AM, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Printing out test numbers frequently exceeds the 80 character limit,<br>
at which point the logger starts writing many copies of the progress<br>
bar instead of overwriting it.  It also prompts people to say things<br>
like "hey, test 8423 is broken!" without realizing that tests are run<br>
in a nondeterministic order, so the number is meaningless.<br>
<br>
The real point was to be able to see when a thread gets stuck - i.e.<br>
one test keeps running forever.  We can easily show this with ASCII<br>
progress spinners.<br>
<br>
Instead of:<br>
<br>
[16816/16822] crash: 21, fail: 82, pass: 12307, skip: 4401, warn: 5 Running Test(s): 16814 16815 16816 16817 16818 16819 16820 16821<br>
<br>
We now get the much more compact:<br>
<br>
[09984/16822] crash: 12, fail: 53, pass: 7314, skip: 2601, warn: 4: ||\-\|/-<br>
<br>
Signed-off-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
---<br>
 framework/log.py | 4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/framework/log.py b/framework/log.py<br>
index caad972..0c9489e 100644<br>
--- a/framework/log.py<br>
+++ b/framework/log.py<br>
@@ -149,12 +149,12 @@ class QuietLog(BaseLog):<br>
         """<br>
         assert self._LOCK.locked()<br>
<br>
-        out = '[{done}/{total}] {status} Running Test(s): {running}'.format(<br>
+        out = '[{done}/{total}] {status}: {running}'.format(<br>
             done=str(self._state['complete']).zfill(self._pad),<br>
             total=str(self._state['total']).zfill(self._pad),<br>
             status=', '.join('{0}: {1}'.format(k, v) for k, v in<br>
                              sorted(self._state['summary'].iteritems())),<br>
-            running=" ".join(str(x) for x in self._state['running'])<br>
+            running=''.join('|/-\\'[x % 4] for x in self._state['running'])<br>
         )<br>
<br>
         self._print(out)<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.1.3<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>
</font></span></blockquote></div><br></div>