-
Notifications
You must be signed in to change notification settings - Fork 3
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
DEBUG log level to export debug files #38
Conversation
Signed-off-by: parvy <pierre.arvy@artelys.com>
unit_Q0[1,g,n], | ||
unit_id[1,g,n], | ||
bus_id[1,n] | ||
> (fileOut); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you should use >> to append lines in the file
unit_id[1,g,n], | ||
bus_id[1,n] | ||
> (fileOut); | ||
printf{LOG_DEBUG} "#variant;num;bus;vRegul;V(pu);targetP(MW);targetQ(Mvar);P(MW);Q(MW);id;bus_id;\n" > (fileOut); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
close the file before, the first printf
Signed-off-by: parvy <pierre.arvy@artelys.com>
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK
Please check if the PR fulfills these requirements
Does this PR already have an issue describing the problem?
Fix #20.
What kind of change does this PR introduce?
The debug files are exported if ampl log level is in DEBUG mode
What is the current behavior?
One debug file is exported at the end of every run. Another is exported only if conditional clause is removed by user in ampl code.
What is the new behavior (if this is a feature change)?
The two debug files are exported if the ampl log level specified by user is DEBUG
Does this PR introduce a breaking change or deprecate an API?
Other information: