Skip to content

Fix test_tools path for detect_numa#67

Merged
dvalinrh merged 1 commit intomainfrom
fix_path
Mar 19, 2026
Merged

Fix test_tools path for detect_numa#67
dvalinrh merged 1 commit intomainfrom
fix_path

Conversation

@dvalinrh
Copy link
Copy Markdown
Contributor

Description

Provides proper path to detect_numa

Before/After Comparison

Before: Failure with message
./run_stream: line 236: /home/ec2-user/test_tools/detect_numa: No such file or directory
(standard_in) 2: syntax error
(standard_in) 1: syntax error
./run_stream: line 314: /home/ec2-user/test_tools//detect_numa: No such file or directory

libgomp: Invalid value for environment variable OMP_NUM_THREADS:

After: Ran successfully and produce results as expected.

Clerical Stuff

This closes #66

Relates to JIRA: RPOPC-888

Testing
Ran streams on test system
Before change, we failed on detect_numa
After change, we produced the expected results

csv file
Array_sizes,36608k,73216k,146432k,292864k,Start_Date,End_Date
Copy,77813,80257,80115,79999,2026-03-19T14:07:14Z,2026-03-19T14:15:58Z
Scale,63074,62899,62624,62493,2026-03-19T14:07:14Z,2026-03-19T14:15:58Z
Add,71302,70123,69988,69861,2026-03-19T14:07:14Z,2026-03-19T14:15:58Z
Triad,70991,70483,70076,69959,2026-03-19T14:07:14Z,2026-03-19T14:15:58Z

@dvalinrh dvalinrh requested a review from frival March 19, 2026 18:05
@dvalinrh dvalinrh self-assigned this Mar 19, 2026
@github-actions
Copy link
Copy Markdown

This relates to RPOPC-888

Copy link
Copy Markdown
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.

LGTM

@frival frival added the group_review_lgtm Indicates approval after a group review meeting label Mar 19, 2026
@dvalinrh dvalinrh merged commit a2efccb into main Mar 19, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

group_review_lgtm Indicates approval after a group review meeting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

issue with path for detect_numa

2 participants