From 9599c867f47a0f462936e2bba01a0402309ec145 Mon Sep 17 00:00:00 2001 From: Yotaro Kubo Date: Thu, 28 Sep 2023 02:46:44 +0900 Subject: [PATCH] Report now includes the results with "-C target-cpu=native" This commit also fixes "reporter.py" that didn't actually rebuild the encoder CLI. --- .cargo/config | 2 ++ report/report.md | 16 ++++++++-------- testtool/reporter.py | 2 +- 3 files changed, 11 insertions(+), 9 deletions(-) create mode 100644 .cargo/config diff --git a/.cargo/config b/.cargo/config new file mode 100644 index 0000000..ddff440 --- /dev/null +++ b/.cargo/config @@ -0,0 +1,2 @@ +[build] +rustflags = ["-C", "target-cpu=native"] diff --git a/report/report.md b/report/report.md index 8099dc2..1aaf525 100644 --- a/report/report.md +++ b/report/report.md @@ -22,15 +22,15 @@ Sources used: wikimedia.i_love_you_california, wikimedia.winter_kiss, wikimedia. ### Average compression speed (inverse RTF) - Reference - - opt8lax: 258.92532958937727 - - opt8: 262.24392534874113 - - opt5: 502.5549809373441 + - opt8lax: 255.90808971990495 + - opt8: 260.20840564877153 + - opt5: 501.4437823135729 - Ours - - default: 149.36375631265756 - - st: 57.107646528588624 - - dmse: 110.75909016976695 - - bsbs: 7.152928413350077 - - mae: 31.28330510072386 + - default: 160.6740906413787 + - st: 73.13241880734284 + - dmse: 107.63095111470871 + - bsbs: 9.577720986818926 + - mae: 27.731935515942386 diff --git a/testtool/reporter.py b/testtool/reporter.py index 7c99787..7e591d5 100644 --- a/testtool/reporter.py +++ b/testtool/reporter.py @@ -221,7 +221,7 @@ def main(): # build logged(subprocess.check_call)( - ["bash", "-c", f"cd {project_root}; cargo build --release"] + ["bash", "-c", f"cd {project_root}/flacenc-bin; cargo build --release"] ) print("Running reference encoder.") ref_run_results = run_encoder(