EmotiBit / ofxEmotiBit

OpenFrameworks library and examples for the EmotiBit
MIT License
34 stars 8 forks source link

Feat advertising options #121

Closed produceconsumerobot closed 2 years ago

produceconsumerobot commented 2 years ago

Description

Added a number of features and fixes to enable greater flexibility with advertising.

Requires

New Features

Fixes

nitin710 commented 2 years ago

Testing

Broadcasting vs Unicasting difference

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
nitin710 commented 2 years ago

Tests still needed to be performed

nitin710 commented 2 years ago

Tests

transmission type result
broadcast not detected
unicast detected
nitin710 commented 2 years ago

Approved for merge