Centennial App for hosting AllJoyn device system bridge by sanjaiganesh · Pull Request #9 · tjaffri/msiot-samples

sanjaiganesh · 2016-08-16T23:16:35Z

Adding Centennial app that hosts an App Service, which in turn hosts DSB.
updated test app to consume app service (optional)
Initialize AlJoyn by calling alljoyn_init(). DSB was crashing, esp in debug mode. This is a mandatory step that was missed.
Ensure Bridge solution builds in 'release' mode, in which centennial app is tested.

For detailed info, refer to README.md in BridgeHost project. Below is the summary.

There are two parts in the centennial app

BridgeHost : This project represents UWP portion fo the app that has hosts the app service, which in turns hosts the bridge (DSB).
BridgeHostStartupTask : This project is a win32 application that runs on startup (logon), starts the app service and holds on to it so that underlying AllJoyn bridge (DSB) is alive.

App service stores info (DeviceInfo) about previously onboarded devices in the application data, organized in the following application data container hierarchy.
This data gets deleted upon uninstall. Ideally, corresponding cloud service should store this information which app can sync to.
That will ensure devices appear on the bus even after uninstall/install and not required to be onboarded again.
The name of the child containers to 'OnboardedDevices' container is generated using (app service)client provided 'Category' (in valueset) concatenated with device ID.
This should change per final design.

OnboardedDevices
- WinkThermoStat:'device id'
  - 'DeviceInfo'
- WinkThermoStat:'device id'
  - 'DeviceInfo'
- VeeraLightBulb:'device id'
  - 'DeviceInfo'
    This data is used when gets activated upon startup (logon) and onboards those devices.

The Ultimate effect is, user sees the devices (on AJ bus), even after restarting the machine. This is the magic we want.
Console or client app can come and onboard new devices and go. But Bridge/App services stays alive.
Startup task has the health check (does every 5 seconds) and starts app service if not UP.

sanjaiganesh · 2016-08-16T23:17:31Z

@tjaffri @jasongin Please take a look. #Resolved

sanjaiganesh · 2016-08-17T00:09:19Z

The main files that you would care are below. Rest are details.

Startup Task: AllJoyn/Platform/BridgeHostStartupTask/Program.cs
App Service: AllJoyn/Platform/BridgeHost/App.xaml.cs

Test app:
AllJoyn/Platform/TestApps/UwpTestApp/MainPage.xaml.cs #ByDesign

jasongin · 2016-08-17T18:00:16Z

AllJoyn/Platform/Bridge.sln

 Microsoft Visual Studio Solution File, Format Version 12.00
-# Visual Studio 14
-VisualStudioVersion = 14.0.24720.0
+# Visual Studio 15


We probably shouldn't force everyone to upgrade to a prerelease version of VS. Or is it needed for some reason? #Closed

Not required. I shall fix that

In reply to: 75173869 [](ancestors = 75173869)

But it opens in VS 14 as well. So no issue as such.

In reply to: 75178273 [](ancestors = 75178273,75173869)

#Resolved

In reply to: 75179512 [](ancestors = 75179512,75178273,75173869)

jasongin · 2016-08-17T18:05:22Z

Update these flavors also? #Closed

Refers to: AllJoyn/Platform/Bridge/ScriptAdapterLib/ScriptAdapterLib.vcxproj:193 in e5c638c. [](commit_id = e5c638c, deletion_comment = False)

jasongin · 2016-08-17T18:09:32Z

  <AdditionalIncludeDirectories>..\..\..\External\boost;..\ScriptAdapterLib;..\ScriptAdapterLib\Deps\inc;..\Common;..\Bridge;$(WindowsSdkDir_10)\Include\$(TargetPlatformVersion)\um\AllJoyn;$(ProjectDir);$(GeneratedFilesDir);$(IntDir);%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>

Fix other flavors also? #Closed

Refers to: AllJoyn/Platform/Bridge/ScriptAdapterLibUW/ScriptAdapterLibUW.vcxproj:140 in e5c638c. [](commit_id = e5c638c, deletion_comment = False)

jasongin · 2016-08-17T18:54:08Z

AllJoyn/Platform/BridgeHost/App.xaml.cs

+                appServiceConnection = appServiceDetails.AppServiceConnection;
+                appServiceConnection.RequestReceived += AppServiceConnection_RequestReceived;
+                appServiceConnection.ServiceClosed += AppServiceConnection_ServiceClosed;
+            }


Consider refactoring all the app service related code into a separate class, maybe called AllJoynAppService. All the event handlers here as well as some other helper methods could be moved over. In my experience App.xaml.cs tends to get very cluttered if you don't take care to delegate responsibilities. #Closed

shall leave it like this for now and consider refactoring later.

In reply to: 75184136 [](ancestors = 75184136)

#ByDesign

In reply to: 75206780 [](ancestors = 75206780,75184136)

jasongin · 2016-08-17T19:33:23Z

🕐

…asks yet to be fixed

sanjaiganesh · 2016-08-19T19:15:50Z

  <AdditionalIncludeDirectories>..\..\..\External\boost;..\ScriptAdapterLib;..\ScriptAdapterLib\Deps\inc;..\Common;..\Bridge;$(WindowsSdkDir_10)\Include\$(TargetPlatformVersion)\um\AllJoyn;$(ProjectDir);$(GeneratedFilesDir);$(IntDir);%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>

#Resolved

In reply to: 240497696 [](ancestors = 240497696)

Refers to: AllJoyn/Platform/Bridge/ScriptAdapterLibUW/ScriptAdapterLibUW.vcxproj:140 in e5c638c. [](commit_id = e5c638c, deletion_comment = False)

sanjaiganesh · 2016-08-19T19:15:55Z

#Resolved

In reply to: 240496420 [](ancestors = 240496420)

Refers to: AllJoyn/Platform/Bridge/ScriptAdapterLib/ScriptAdapterLib.vcxproj:193 in e5c638c. [](commit_id = e5c638c, deletion_comment = False)

Sanjai Ganesh Jeenagiri added 2 commits August 16, 2016 16:03

Centennial App for hosting AllJoyn device system bridge

5a122cb

minor corrections to readme.md

74f6aef

updating readme.md with list of files that have the implementation

e5c638c

tjaffri assigned jasongin Aug 17, 2016

jasongin reviewed Aug 17, 2016
View reviewed changes

Sanjai Ganesh Jeenagiri added 3 commits August 18, 2016 11:50

addressing some of the review comments.

0e2cff4

Now builds for Arm/x86 debug/release as well. BridgeHost post build t…

ee30719

…asks yet to be fixed

Fixed post build tasks as well

12fb1bd

Conversation

sanjaiganesh commented Aug 16, 2016

There are two parts in the centennial app

Uh oh!

sanjaiganesh commented Aug 16, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

sanjaiganesh commented Aug 17, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jasongin Aug 17, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sanjaiganesh Aug 17, 2016

Choose a reason for hiding this comment

Uh oh!

sanjaiganesh Aug 17, 2016

Choose a reason for hiding this comment

Uh oh!

sanjaiganesh Aug 19, 2016

Choose a reason for hiding this comment

Uh oh!

jasongin commented Aug 17, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jasongin commented Aug 17, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jasongin Aug 17, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sanjaiganesh Aug 17, 2016

Choose a reason for hiding this comment

Uh oh!

sanjaiganesh Aug 19, 2016

Choose a reason for hiding this comment

Uh oh!

jasongin commented Aug 17, 2016

Uh oh!

sanjaiganesh commented Aug 19, 2016

Uh oh!

sanjaiganesh commented Aug 19, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

sanjaiganesh commented Aug 16, 2016 •

edited

Loading

sanjaiganesh commented Aug 17, 2016 •

edited

Loading

jasongin Aug 17, 2016 •

edited

Loading

jasongin commented Aug 17, 2016 •

edited

Loading

jasongin commented Aug 17, 2016 •

edited

Loading

jasongin Aug 17, 2016 •

edited

Loading