Skip to content

Commit

Permalink
Fix summary output
Browse files Browse the repository at this point in the history
  • Loading branch information
goodsong81 committed Apr 12, 2024
1 parent ac640de commit 84e4d5a
Show file tree
Hide file tree
Showing 3 changed files with 1,804 additions and 1,804 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/perf_benchmark.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -141,15 +141,15 @@ jobs:
--num-repeat ${{ inputs.num-repeat }}
--num-epoch ${{ inputs.num-epoch }}
--eval-upto ${{ inputs.eval-upto }}
--summary-file .tox/perf-benchmark-summary.csv
--summary-file .tox/perf-benchmark-summary.xlsx
--mlflow-tracking-uri ${{ vars.MLFLOW_TRACKING_SERVER_URI }}
--user-name ${{ github.triggering_actor }}
--otx-ref ${{ inputs.otx-ref }}
- name: Upload test results
uses: actions/upload-artifact@5d5d22a31266ced268874388b861e4b58bb5c2f3 # v4.3.1
with:
name: perf-benchmark-${{ matrix.task-short }}
path: .tox/perf-*.csv
path: .tox/perf-benchmark-*.*
# Use always() to always run this step to publish test results when there are test failures
if: ${{ always() }}

Expand Down
2 changes: 1 addition & 1 deletion tests/perf/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -384,7 +384,7 @@ def fxt_benchmark_summary(
if fxt_summary_file.suffix != ".csv":
print(f"{fxt_summary_file.suffix} output is not supported.")
fxt_summary_file = fxt_summary_file.with_suffix(".csv")
summary_results.to_csv(fxt_summary_file, index=False)
summary_results.to_csv(fxt_summary_file)
print(f" -> Saved to {fxt_summary_file}.")

if fxt_mlflow_client:
Expand Down
Loading

0 comments on commit 84e4d5a

Please sign in to comment.