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

Print to file in debug mode #135

Merged
merged 2 commits into from
Dec 9, 2024

Conversation

hecmas
Copy link
Contributor

@hecmas hecmas commented Dec 9, 2024

This pull request introduces a feature to optionally print debug information to a file instead of standard output.

Key changes include:

Command Structures:

  • Added a new boolean flag print_to_file to ProveCmd and VerifyConstraintsCmd structs to enable printing debug information to a file.

Mode Initialization:

  • Updated the StdMode struct and its constructor to include the print_to_file flag.
  • Modified StdMode::new calls to accommodate the new print_to_file parameter.

Debug Printing:

  • Enhanced the print_debug_info function to handle writing to a file if the print_to_file flag is set, including creating the necessary directory and file.
  • Updated calls to print_debug_info in StdProd and StdSum implementations to pass the print_to_file flag.

These changes ensure that debug information can be optionally redirected to a file, improving flexibility and usability in different debugging scenarios.

@hecmas hecmas requested a review from zkronos73 December 9, 2024 10:14
@hecmas hecmas self-assigned this Dec 9, 2024
@cla-bot cla-bot bot added the cla-signed label Dec 9, 2024
@hecmas hecmas mentioned this pull request Dec 9, 2024
16 tasks
@hecmas hecmas force-pushed the feature/print-file-debug branch from aa5c592 to 812a233 Compare December 9, 2024 10:18
@hecmas hecmas merged commit 0c211c3 into feature/global-bus-debug Dec 9, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant