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

Timing test #103

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Timing test #103

wants to merge 9 commits into from

Conversation

meredithmhu
Copy link
Contributor

@meredithmhu meredithmhu commented May 18, 2024

(Meredith) Created a time.py script for automatic push-button functionality for compiling and running Caiman programs for the purposes of gathering timing data. Features include:

Format of the commands is python [relative_path_to_time.py] [command] [relative_path_to_file] [NUM_ITERS]

  1. COMPILATION OR RUNNING WITH CONSOLE-PRINTED SUCCESS & RUNTIME: ability to compile or run any .cair file from anywhere in the Caiman directory. (A note: this script is located in caiman/caiman-test.) A message of success will print to the console if your file successfully compiles, or runs, along with the resulting compile-time or runtime of the test.
  2. MULTIPLE ITERATIONS SUPPORTED: script can run multiple iterations of any command to compile or run some particular test.
  3. CSV GENERATION: the data generated from a command that requests multiple iterations of a command to be performed are automatically saved to caiman-test/test_data/your-file-name_command.csv. If the file has been tested before, the data in this file is automatically overwritten.
  • additionally, these csv files are in the gitignore. They do not get pushed to the main Caiman branch.

Copy link
Contributor

@Checkmate50 Checkmate50 left a comment

Choose a reason for hiding this comment

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

Looks great to me! Could the CSV files be removed from the commit before I accept?

@meredithmhu
Copy link
Contributor Author

@Checkmate50 Hey, sorry this came a few months late, but I removed the CSV files and you should be able to accept now. Let me know if you need anything else to be changed!

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