-
Notifications
You must be signed in to change notification settings - Fork 54
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
Make "declared function" printouts debug mode #472
base: main
Are you sure you want to change the base?
Conversation
CodSpeed Performance ReportMerging #472 will not alter performanceComparing Summary
|
I'm not sure if this should be in the |
Is this documented anywhere? Why would this be the case? |
@yihozhang is this for the web demo? Or for sidecar egglog? |
Sidecar egglog only uses stdout for communication. |
Racket's REPL, for example, doesn't print out something on each command, it just runs it. It's pretty verbose to print on each command unless you need that interactivity for sidecar egglog. |
The eggcc project relies on info printing, and the declared function print outs are becoming super verbose (one per variable, which we generate tens of thousands of)
This PR makes these prints debug instead of info
It also slightly cleans up a confusing error message in the extractor