rosjava / android_apps

Applications built on top of the rosjava android libraries.
133 stars 112 forks source link

Platform info service in AppManager #12

Closed stonier closed 11 years ago

stonier commented 11 years ago

Add this to the app manager so we can start retrieving icons and remove the redundant parameter requirements.

stonier commented 11 years ago

Quite a few commits went in to this, so can't track them here.

The core problem was calling platform info from the master chooser activity (not a rosactivity). You can do that rather simply though just constructing the nodemainexecutor and nodeconfiguration with defaults and running the calls as per normal.

I split this into its own class (outside of the app manager class). Much easier to follow and read than the AppManager borg, especially given that AppManager class' instances can only execute once anyway.

stonier commented 11 years ago

Also removed all parameter usage in the robot remocon and also in the paired master script.