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.

Sign up for GitHub

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

Jump to bottom

Is success variable required in the setNoiseProfile service? #304

Open
etsardou opened this issue May 16, 2016 · 4 comments
Open

Is success variable required in the setNoiseProfile service? #304

etsardou opened this issue May 16, 2016 · 4 comments
Labels
question
Milestone
FutureWork

Comments

@etsardou
Copy link
Member

https://github.com/rapp-project/rapp-platform/blob/devel/rapp_platform_ros_communications/srv/AudioProcessing/AudioProcessingSetNoiseProfileSrv.srv#L11
@etsardou etsardou added this to the v0.6 milestone May 16, 2016
@etsardou
Copy link
Member Author

Same for here

@etsardou
Copy link
Member Author

@klpanagi do you check the success variable in the web services?

@klpanagi
Copy link
Contributor

The success property is parsed, though it is not needed as we parse the error property of the message too.

@etsardou
Copy link
Member Author

Ok, will remove it in 0.7

@etsardou etsardou modified the milestones: v0.7, v0.6 May 20, 2016
@etsardou etsardou modified the milestones: FutureWork, v0.7 Nov 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question
Projects
None yet
Milestone
FutureWork
Development

No branches or pull requests
2 participants
@klpanagi @etsardou