Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix missing metadata header. #30

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix missing metadata header. #30

wants to merge 3 commits into from

Conversation

dvalinrh
Copy link
Contributor

@dvalinrh dvalinrh commented Feb 18, 2025

Description

It places the system metadata in the csv results file, currently missing

Before/After Comparison

Before change:
Benchmarks:Base copies:Base Run Time:Base Rate
500.perlbench_r:4:475:13.4
502.gcc_r:4:346:16.4
505.mcf_r:4:396:16.3
520.omnetpp_r:4:668:7.85

After change
# Test general meta start
# Test: speccpu2017
# Results version: 118
# Host: m6a.xlarge
# Sys environ: aws
# Tuned: virtual-guest
# OS: 5.14.0-362.18.1.el9_3.x86_64
# Numa nodes: 1
# CPU family: AMD EPYC 7R13 Processor
# Number cpus: 4
# Memory: 15720220kB
# Test general meta end
Benchmarks:Base copies:Base Run Time:Base Rate
500.perlbench_r:4:475:13.4
502.gcc_r:4:346:16.4
505.mcf_r:4:396:16.3
520.omnetpp_r:4:668:7.85

Clerical Stuff

This closes Issue #29

Relates to JIRA: RPOPC-<TICKET_NUMBER>

@dvalinrh dvalinrh requested a review from a team February 18, 2025 21:33
Copy link
Contributor

@frival frival left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As mentioned in slack, can you fix the formatting on the initial comment? Also it looks like two spurious whitespace lines. Any other benchmarks missing header info?

@dvalinrh
Copy link
Contributor Author

Comment section above fixed.
This is the only workload I see the meta header missing in.

@dvalinrh dvalinrh requested a review from frival February 19, 2025 12:10
@@ -232,7 +232,7 @@ done
# clone the repo.
#
if [ ! -d "test_tools" ]; then
git clone $tools_git ${curdir}/test_tools
git clone $tools_git test_tools
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrong, use $curdir not pwd.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants