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

Adding total runtime print message #67

Closed
wants to merge 1 commit into from
Closed

Conversation

ZachJHansen
Copy link
Collaborator

Closes #64

@teiesti
Copy link
Collaborator

teiesti commented Feb 23, 2024

I am against this change, at least in its current form. My reasons are:

  1. You can always time a command on the runtime with time <command>, giving you not only the walltime but also the respective CPU times.
  2. When translating, the output is currently your expected program or theory without any additional elements. Adding the time at the end makes it much more difficult to chain various translations, e.g., anthem translate --with tau_star foo.lp | anthem translate --with gamma. Of course, we can circumnavigate this concern by printing additional information to stderr.

@teiesti teiesti added C-feature-requested Category: Requested feature P-low Priority: Low A-command-line Area: Command-line interfaxe E-easy Experience: Easy labels Feb 23, 2024
@ZachJHansen ZachJHansen deleted the zach/time branch February 23, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-command-line Area: Command-line interfaxe C-feature-requested Category: Requested feature E-easy Experience: Easy P-low Priority: Low
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Display system runtime
2 participants