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

Feat advertising options #121

Merged
merged 19 commits into from
May 13, 2022
Merged

Feat advertising options #121

merged 19 commits into from
May 13, 2022

Conversation

src/EmotiBitWiFiHost.h Outdated Show resolved Hide resolved
src/EmotiBitWiFiHost.h Show resolved Hide resolved
src/EmotiBitWiFiHost.cpp Show resolved Hide resolved
src/ofxEmotiBitVersion.h Outdated Show resolved Hide resolved
EmotiBitOscilloscope/src/ofApp.cpp Show resolved Hide resolved
src/EmotiBitWiFiHost.cpp Show resolved Hide resolved
@nitin710
Copy link
Collaborator

nitin710 commented May 11, 2022

Testing

Broadcasting vs Unicasting difference

  • Testing Notes
    • Case 1:

      • 3 Oscilloscopes(v1.4.4-feat-advertisingOptions.6) open in Windows-dev(running app-debug.exe) with broadcast enabled
      • 1 oscilloscope(v1.4.4) open in deploy
    • Case 1 observations using unicast

      • The 3 oscilloscopes in dev are connected continuously and streaming without problem
      • When the oscilloscope(v1.4.4) was started on deploy, it did not show up any emotibits on the network(for some time). Then all EmotiBits shoed up as expected.
      • Momentarily, all 3 v1.4.4-feat-advertisingOptions.6 oscilloscopes dropped their connections to EmotiBits. I wonder is v1.4.4 coming on the network had an effect?
      • V1.4.4 dropped connection to emotibit after 2 minutes. But v1.4.4-feat-advertisingOptions.6 is continuously connected and streaming well
    • Case 2:

      • 3 - 3 Oscilloscopes(v1.4.4-feat-advertisingOptions.6) open in Windows-dev(running app-debug.exe) with unicast enabled
      • 1 oscilloscope(v1.4.4) open in deploy
    • Case 2 observations

      • v1.4.4-feat-advertisingOptions.6
      • There is a lag in picking up emotibits on the network
      • more frequent dropping
      • more perceived lag in streaming
      • lag while connecting to EmotiBits
      • v1.4.4
        • has not dropped an EmotiBit since the beginning
        • Looks to be streaming OK
    • Case 3:

      • v1.4.4-feat-advertisingOptions.6
        • Started 2 oscilloscopes in unicast mode
        • changed transmission -> broadcast in JSON file
        • Started another instance of oscillosocpe
    • Case 3 observations:

      • After ~10 mins, only the oscilloscope using broadcast was connected and streaming data from Emotibits. The other EmotiBits seemed to have disconnected from their emotibits
Transmission Network Name Router model # # EmotiBits on network # Oscilloscopes on network Observation duration # drops # disconnects
broadcast EmotiNet AC750 6 3x 1.4.4-feat-advertisingOptions.6
1x v1.4.4
30 mins 1-2(on 1.4.4-feat-advertisingOptions.6)
multiple - v1.4.4
1 - 1.4.4-feat-advertisingOptions.6
multiple -v1.4.4
unicast EmotiNet AC750 6 3x 1.4.4-feat-advertisingOptions.6
1x v1.4.4
10 mins few(on 1.4.4-feat-advertisingOptions.6)
none - v1.4.4
multiple - 1.4.4-feat-advertisingOptions.6
multiple -v1.4.4

Unicast range

EmotiBits are allocated IPs starting from 100.

Test Case Output
Toggling both broadcast and unicast OFF ✔️ All EmotiBits are found
If both broadcastunicast is used by default, (according to the code as is)
image
Unicast range does not include EmotiBit IPs ✔️ No emotibits found
image
Unicast range extended to cover only 1 Emotibit ✔️ Only 1 EmotiBit found
image
Unicast range extended to cover only 2 EmotiBits ✔️ 2 EmotiBits found
image

Whitelist vs blacklist

Available networks(adapters) on the computer:

All Network(s): [192.168.40.*] [192.168.193.*] [192.168.1.*]
Emotibits on network: 192.168.1.*
Case output
Include list is not empty and does not contain EmotiBit network ✔️ EmotiBits not detected
image
Include list = emotibit network ✔️ All EmotiBits found
image
Include list contains > 1 networks, and EmotiBit network is a part of the list ✔️ All EmotiBits on network found
image
EmotiBit network added to the exclude list ✔️ EmotiBits not found on network
image
image

src/EmotiBitWiFiHost.cpp Outdated Show resolved Hide resolved
src/EmotiBitWiFiHost.cpp Outdated Show resolved Hide resolved
@nitin710 nitin710 added the duplicate This issue or pull request already exists label May 12, 2022
@nitin710
Copy link
Collaborator

nitin710 commented May 12, 2022

Tests still needed to be performed

src/EmotiBitWiFiHost.cpp Outdated Show resolved Hide resolved
src/EmotiBitWiFiHost.cpp Outdated Show resolved Hide resolved
@nitin710 nitin710 removed the duplicate This issue or pull request already exists label May 12, 2022
@nitin710
Copy link
Collaborator

nitin710 commented May 12, 2022

Tests

  • ✔️ Oscillosocpe on iPhone hotspot
transmission type result
broadcast not detected
unicast detected

@nitin710
Copy link
Collaborator

Approved for merge

@nitin710 nitin710 merged commit 00eac10 into master May 13, 2022
@nitin710 nitin710 deleted the feat-advertisingOptions branch May 13, 2022 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment