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

Refactor CLI parsing for synthesis entrypoint #6

Merged
merged 3 commits into from
May 2, 2024
Merged

Conversation

shadaj
Copy link
Member

@shadaj shadaj commented May 1, 2024

Related to #3

This pull request simplifies the CLI parsing logic for the synthesis entrypoint in tests/synthesize_crdt.py and enhances the user experience by utilizing the argparse library.

  • Replaces manual sys.argv parsing with structured command-line options using argparse, making the script more user-friendly and maintainable.
  • Introduces a main function to encapsulate the script logic, improving readability and structure.
  • Adds argument parsing with options for synthesis mode, benchmark selection, and additional flags (--fixed, --first, --repeat), allowing users to easily specify their synthesis requirements.
  • Implements help messages for each command and argument, automatically generated by argparse, providing users with clear guidance on how to use the script.
  • Updates the README.md to reflect the new argument structure, ensuring documentation is consistent with the code changes.

For more details, open the Copilot Workspace session.

@shadaj shadaj linked an issue May 1, 2024 that may be closed by this pull request
@shadaj shadaj closed this May 1, 2024
@shadaj shadaj deleted the shadaj/cli-refactor branch May 1, 2024 19:45
@shadaj shadaj restored the shadaj/cli-refactor branch May 1, 2024 19:47
@shadaj shadaj reopened this May 1, 2024
@shadaj shadaj force-pushed the shadaj/cli-refactor branch from 2631982 to 7c06b77 Compare May 1, 2024 19:47
@shadaj shadaj merged commit b5c0e0c into main May 2, 2024
3 checks passed
@shadaj shadaj deleted the shadaj/cli-refactor branch May 2, 2024 05:16
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.

Rewrite CLI parsing logic for synthesis entrypoint
1 participant