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

[Feature Request] Disable output from brick connect #17

Closed
GoogleCodeExporter opened this issue Oct 10, 2015 · 2 comments
Closed

[Feature Request] Disable output from brick connect #17

GoogleCodeExporter opened this issue Oct 10, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

Describe the new feature more precisely.

Add a parameter to find_one_brick and find_bricks to disable preliminary error 
output. (Such as USB not available) Of course, keep the catchall that throws an 
official error. Another option would be to have a global nxt module variable 
that specifies if output is requested.


Why do you think it's important?
It would allow automated programs to look more professional and output their 
own error codes.

Any ideas on how to implement it?
Just add a parameter output=True to the functions, and have find_one_brick pass 
to find_bricks. All outputs would then be wrapped with an if block.

Please provide any additional information below.
This is a quick and low priority change that does not change the overall 
function of the program at all.

Original issue reported on code.google.com by Zonedab...@gmail.com on 11 Nov 2010 at 2:58

@GoogleCodeExporter
Copy link
Author

Good idea; I will be looking at this and probably set it so that it only prints 
it if a "debug" option is passed (and perhaps add more messages as well).

Original comment by marcus@wanners.net on 11 Nov 2010 at 3:06

  • Changed state: Accepted
  • Added labels: Component-UI, OpSys-All, Priority-Low, Series-2.x
  • Removed labels: Priority-Medium

@GoogleCodeExporter
Copy link
Author

Fixed in r269. Please verify that this is the behavior you were requesting.

Original comment by marcus@wanners.net on 3 Jan 2011 at 7:31

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant