-
Notifications
You must be signed in to change notification settings - Fork 712
add wifi7 320Mhz support #511
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
add wifi7 320Mhz support #511
Conversation
|
add to context :
|
|
Thank you for creating a PR.
Very Strange, it could be related to Ubiquiti Unifi U7 Pro and/or Sony Xperia 1 VI Can you please provide some raw scan results for 320MHz and 160MHz? Reference docs: |
320MHZ channel 31 (control 5)320MHZ channel 95 (control 85)160MHZ channel 79(control 85)160MHZ channel 15(control 5)160MHZ channel 47(control 53)https://www.rfwireless-world.com/calculators/WiFi-7-channel-number-to-center-frequency-conversion.html
|
|
Thanks for the information 320MHz channel 31 (control 5) - this one is a match 320MHz channel 95 (control 85) - this one does not match - channel 95 - frequency: 6425 160MHz channel 79 (control 85) - this one is a match 160MHz channel 15 (control 5) - this one is a match 160MHz channel 47 (control 53) - this one is a match ! One data sample data is strange, the rest do match
Sets are overlapping now, that the reason for the failures. |
app/src/main/kotlin/com/vrem/wifianalyzer/wifi/band/WiFiChannelsGHZ6.kt
Outdated
Show resolved
Hide resolved
this one is not clear , since the Unifi network only allow user to select control channel , which is 85 , so I'm not very sure it's on 95 (from the chart it seems), I think we need other people with other hardware to test |
|
Is it possible to re-test 320MHz with control channel 85, based on scan result center channel needs to be 63? |
|
FYI and this is raw scan results from wifi analyzer debug mode |
app/src/test/kotlin/com/vrem/wifianalyzer/wifi/band/WiFiChannelsGHZ6Test.kt
Show resolved
Hide resolved
|
Thanks for the information |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #511 +/- ##
=========================================
Coverage 95.09% 95.10%
Complexity 915 915
=========================================
Files 131 131
Lines 2712 2715 +3
Branches 208 209 +1
=========================================
+ Hits 2579 2582 +3
Misses 48 48
Partials 85 85 🚀 New features to boost your workflow:
|
a962728
into
VREMSoftwareDevelopment:main
|
Hi @hsinyu-chen, The following changes have been committed to the main branch:
Would it be possible to conduct testing for the 5/6 GHz frequency bands and 160/320 MHz channel widths? Your comments or recommendations on the changes would be highly appreciated. Additionally, it's possible to obtain the APK without performing a local build. The artifacts are available via this workflow. Thanks |
|
will report 160Mhz result later, need wait everyone go sleep so I can change the AP setting without effect them |
|
Probably need more raw scan results from wifi analyzer debug mode for 160 and 320
Yes, if more than 1 AP is displayed close to each other, by clicking it will iterate over them as pop-up |
|
??? Maybe for 320 need to look at both center0 and center1 and use some kind of formula Probably having raw results for both 160 and 320 will help Again thanks for your help |
can we have something that can get raw result in the app without local debugger attached? |
|
Please try the following:
This may not give all details as available in ScanResult Another Developer Options link: https://developer.android.com/studio/debug/dev-options |
I tried , but not detail enough for us , will get ScanResult later, maybe cloud add |
|
@VREMSoftwareDevelopment I have confirm the ScanResult is same as last time I proved, I have dig this and I think maybe the issue is on the Unifi side(wifi man display logic) this is linux kernal code for calculate wifi channel wireless/util.c#L136 and I port it to javascript , feed data from my scanresult, ported code gist and the output so the channel (center freq) is all matched other than so I think we are all good here Add to this |
|
Looks like 160 work good center1 is actually center Will post when new hack is available |
|
Hi @hsinyu-chen, I wanted to inform you that new code to calculate the center for 160/320 MHz widths has been committed to the main branch. Your feedback or recommendations on these changes would be greatly appreciated. Additionally, you can obtain the APK without performing a local build. The artifacts are available through this workflow. Looking forward to your insights! Best regards |
|
@VREMSoftwareDevelopment I test the app and look the code , I think it's too hacky and may not present actually situation. the first one looks "right" but it is not accurate , the 85 should be able to use 320MHZ-1 channel 63 or 320MHZ-0 95 the second one have same issue ,on the bad way , the beacon channel 53 can use 320MHZ-1 channel 63 , but due to current center calculation , it look like it is using 31 exclusively IMO for best present wifi7 320Mhz channel usage, maybe it only display 20MHZ beacon channel as solid color , and a dash line border for possible channels? |
This is not trivial to do. |
















Thanks for sending a pull request!
How to submit a pull request
What does this implement/fix? Please describe.
Does this close any currently open issues?
Additional context
centerFreq1, tested withSony Xperia 1 VIbut this against api description , I'm not sure it can be usedcenterFreq1, add 3rd args to calculateCenter funcWhere has this been tested?
Wifi 7 AP : Ubiquiti Unifi U7 Pro
Client Device : Sony Xperia 1 VI