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

[concert_admin_app] cannot access parameter of concert service in remote server #170

Open
jihoonl opened this issue Dec 1, 2014 · 5 comments

Comments

@jihoonl
Copy link
Collaborator

jihoonl commented Dec 1, 2014

The current concert_admin_app is only able to configure concert service parameters only if concert is running on local machine.

If concert is running in remote pc, it cannot update service parameters. enable/disable works.

Local concert

  • enable/disable
  • parameter configuration

Remote Concert

  • enable/disable
  • parameter configuration
@jihoonl
Copy link
Collaborator Author

jihoonl commented Dec 1, 2014

Probably, concert_service_manager should be a bridge node to update service parameters.

@dwlee
Copy link
Collaborator

dwlee commented Jan 7, 2015

The existing admin app can only change local file regarding service profile. So, this issue have be occurred when user used admin app in remote. I suggest improved systems, service manager and admin app for solving this issue as following figure.

Relationship between Service manager and Admin app in remote

  • API change of list for including real data about service profile (ex. parameters, interactions, icon...)
    • For displaying up to date service profile information in admin app launched remote
    • Admin app show only service profile information using data from list topic
    • proposed message
  • Addition new ros service for update service profile in remote.
    • For changing file of service profile via other node including admin app.
  • Implementation of File checker about file of service profile in service manager
    • For keeping up to date service profile and publishing it through list topic.
    • Service profile files can be changed by two cases. One is user modifies it, directly. The other is via admin app. So we use file checker because detected modification of service profile by two case.

@dwlee
Copy link
Collaborator

dwlee commented Jan 9, 2015

We discussed above suggestion and changed one thing about file checker.

  • Cache checker instead of file checker
    • If the cache file already exists, service manager read it instead of creating new cache.

@dwlee
Copy link
Collaborator

dwlee commented Jan 12, 2015

This comment has been migrated to robotics-in-concert/rocon_concert#254

@jihoonl
Copy link
Collaborator Author

jihoonl commented Jan 12, 2015

Solution service cache creation will be discussed in robotics-in-concert/rocon_concert#254

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

No branches or pull requests

2 participants