-
Notifications
You must be signed in to change notification settings - Fork 67
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 the Quiet Mode Actually Quiet #364
Comments
@DimitrisMantas , Can you take a look at the code and see if there is an easy way to turn this off You can make a pull request or let us know your findings |
Hi, @santoshphilip, Thank you for immediate response! I'll be sure to take a look at the issue. |
I did some testing with the code in your pull request. I discovered that the older versions of E+ (8.5 for instance) do not print out the line You opened this issue for a specific use case that you had. Maybe we have a section called Running verbose, quiet or silent - is it a submarine ?explaining why we need silent. Once we wrap up this issue I'll open a new issue to update to the user documentation with the new section |
Sure, I completely agree with updating the documentation to reflect the proposed changes and can definitely write something up. Where should I post it? |
Introduce Silent Verbose Mode (Issue #364)
@DimitrisMantas I referred to this issue in unmethours.com You can respond in unmethours.com with an answer (solution: is your pull request) Shall we close this issue. (You should do the closing). A new issue can be opened if documentation is critical |
Alright. I'll be closing the issue and will keep a heads up to write some relevant documentation in case the proposed changees make it into the main branch at some point. |
Once it gets into the develop branch, it will get into the master branch for sure :-)
On Saturday, December 4, 2021, 08:43:19 AM PST, Dimitris Mantas ***@***.***> wrote:
Alright. I'll be closing the issue and will keep a heads up to write some relevant documentation in case the proposed changees make it into the main branch at some point.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
|
@DimitrisMantas , I am glad to see that the changes you made are useful to other people, See this question on https://stackoverflow.com about python lists comprehension. There are comments and answers. I takes time to compose a good answer. You can take your time composing a good response/answer. You can also edit it later to make it better. At the moment, there is enough information for the person to solve his problem, so there is no hurry to post the answer |
Hello everyone,
Whenever I use run(..., verbose="q"), the output to the console is actually
EnergyPlus Completed Successfully.
This is potentially unwanted behavior in cases where one might use eppy in conjunction with another program that also prints stuff to the console. For example, if you're running E+ multiple times with different .IDF files to optimize a specific variable, then you might need to print relevant metrics at specific time intervals. However, if eppy is constantly printing to the console, the stuff that actually matters is obscured by the
EnergyPlus Completed Successfully.
messages.Thanks.
The text was updated successfully, but these errors were encountered: