Skip to content

Conversation

jaja360
Copy link
Collaborator

@jaja360 jaja360 commented Jul 15, 2025

This PR introduces enhancements to the scripts directory, focusing on documentation updates, improved AWS testing capabilities, and modifications to the generate_multiple_tables.py script for better output handling and environment configuration.

Documentation Updates:

  • Updated scripts/README.md to include instructions for generating multiple LaTeX tables and running tests on Amazon AWS. Added details about the generate_multiple_tables.py and aws_tests.py scripts, including their functionality and prerequisites.

AWS Testing Enhancements:

  • Modified scripts/aws_tests.bash to explicitly set the CC environment variable for both gcc and clang builds, ensuring proper compiler configuration during benchmarking.

Script Improvements:

  • Updated get_cpu_model() in scripts/generate_multiple_tables.py to set the LANG environment variable to "C" for consistent output across different locales.
  • Enhanced process_job() in scripts/generate_multiple_tables.py to generate both .tex and .raw output files, providing flexibility for downstream processing.

@jaja360 jaja360 requested a review from lemire July 15, 2025 16:19
Copy link
Member

@lemire lemire left a comment

Choose a reason for hiding this comment

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

😄

@jaja360 jaja360 merged commit 201a466 into master Jul 15, 2025
8 checks passed
@jaja360 jaja360 deleted the tweak_scripts branch July 15, 2025 23:00
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