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

Remove a print statement which is not controlled by any flag. #132

Merged
merged 1 commit into from
Nov 2, 2018

Conversation

tujian
Copy link
Contributor

@tujian tujian commented Nov 2, 2018

This print statement is removed because:

  1. It is not controlled by any flag. Maybe it was written for debug purpose.

  2. More detailed could be printed by mosek. And the printed status is not a temporary variable, instead it is already returned as a value in the dict. Any print statement could be conducted with that dict.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 55.479% when pulling 2cc837b on tujian:remove-uncontrolled-print into afc6d95 on cvxopt:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 55.479% when pulling 2cc837b on tujian:remove-uncontrolled-print into afc6d95 on cvxopt:master.

@martinandersen
Copy link
Contributor

@tujian, thanks for pointing this out.

@martinandersen martinandersen merged commit 2bb3a42 into cvxopt:master Nov 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants