Announcement

Collapse
No announcement yet.

sunflow test parser bug

Collapse
X
  • Filter
  • Time
  • Show
Clear All
new posts

  • sunflow test parser bug

    I've just been running the universe test on one of my machines and I've noticed a bug in the parser which is similar to the scale problem identified in the md-gromacs test-set.

    On fast machines, the result given isn't in seconds anymore, it's in milliseconds.
    This leads to some stupid results for fast machines like the one I tested on. It reported times of 7000ms which the parser interprets as 7000s.

    I'm looking into the parser now to work out how best to fix it.

  • #2
    OK, this patch should fix the parser:

    Code:
    diff --git a/pts/test-resources/sunflow/parse-results.php b/pts/test-resources/sunflow/parse-results.php
    index d2332d2..f4a3c1e 100644
    --- a/pts/test-resources/sunflow/parse-results.php
    +++ b/pts/test-resources/sunflow/parse-results.php
    @@ -2,6 +2,11 @@
    
     $BENCHMARK_RESULTS = substr($argv[1], strpos($argv[1], "Average:"));
     $BENCHMARK_RESULTS = substr($BENCHMARK_RESULTS, 0, strpos($BENCHMARK_RESULTS, "\n"));
    -echo trim(substr($BENCHMARK_RESULTS, strrpos($BENCHMARK_RESULTS, ':') + 1));
    +preg_match( '/([0-9\.:]*)(.{0,2})/', trim(substr($BENCHMARK_RESULTS, strrpos($BENCHMARK_RESULTS, ':') + 1)), $match);
    +
    +if($match[2] == 'ms')
    +       echo $match[1] / 1000;
    +else
    +       echo $match[1];
    
     ?>

    Comment


    • #3
      Fixed in git, thanks!
      Michael Larabel
      http://www.michaellarabel.com/

      Comment

      Working...
      X