Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(163)

Issue 329500043: Wi-Fi adjacent channel modeling through transmit spectrum mask (OFDM)

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 months ago by Rediet
Modified:
3 days, 21 hours ago
Reviewers:
S. Deronne, skalpaiit, micha.rademacher, michael.rademacher, Nicola Baldo
CC:
ns-3-reviews_googlegroups.com
Visibility:
Public.

Description

This patch models the transmit spectrum mask defined for OFDM-based Wi-Fi standards (within the WifiSpectrumValueHelper) so as to be able to model adjacent channel interference. It takes ns-3-dev revision 13114:ba42ffbfbe1e (Oct 12 2017) as reference. Any suggestions / reviews would be much appreciated.

Patch Set 1 : Diff with ns-3-dev rev:13114 #

Total comments: 13

Patch Set 2 : Handle review comments and add unit tests #

Total comments: 58

Patch Set 3 : Patch Set 3: Handle Sebastien's comments on 2nd patch #

Total comments: 2

Patch Set 4 : Handle Sebastien's and S.Kalpalatha's comments concerning previous patchset #

Patch Set 5 : Correct channel width and frequency for non-HT OFDM PSD #

Patch Set 6 : Adapting channel width following Sebastien's recommendations in Bug 2483 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1810 lines, -236 lines) Patch
M examples/wireless/wifi-spectrum-saturation-example.cc View 1 2 3 4 1 chunk +2 lines, -2 lines 0 comments Download
M src/spectrum/model/multi-model-spectrum-channel.h View 1 2 3 4 5 2 chunks +11 lines, -0 lines 0 comments Download
M src/spectrum/model/multi-model-spectrum-channel.cc View 1 2 3 4 5 2 chunks +8 lines, -1 line 0 comments Download
M src/spectrum/model/wifi-spectrum-value-helper.h View 1 2 3 3 chunks +65 lines, -1 line 0 comments Download
M src/spectrum/model/wifi-spectrum-value-helper.cc View 1 2 3 4 14 chunks +287 lines, -211 lines 0 comments Download
A src/wifi/examples/wifi-trans-example.cc View 1 1 chunk +311 lines, -0 lines 0 comments Download
A src/wifi/examples/wifi-trans-example.sh View 1 1 chunk +145 lines, -0 lines 0 comments Download
M src/wifi/examples/wscript View 1 chunk +4 lines, -0 lines 0 comments Download
M src/wifi/model/constant-rate-wifi-manager.cc View 1 2 3 4 5 1 chunk +2 lines, -2 lines 0 comments Download
M src/wifi/model/ideal-wifi-manager.cc View 1 2 3 4 5 1 chunk +1 line, -1 line 0 comments Download
M src/wifi/model/minstrel-ht-wifi-manager.cc View 1 2 3 4 5 2 chunks +2 lines, -2 lines 0 comments Download
M src/wifi/model/spectrum-wifi-phy.h View 1 2 3 4 2 chunks +11 lines, -1 line 0 comments Download
M src/wifi/model/spectrum-wifi-phy.cc View 1 2 3 4 4 chunks +24 lines, -9 lines 0 comments Download
M src/wifi/model/wifi-remote-station-manager.h View 1 2 3 4 5 1 chunk +10 lines, -0 lines 0 comments Download
M src/wifi/model/wifi-remote-station-manager.cc View 1 2 3 4 5 6 chunks +20 lines, -5 lines 0 comments Download
M src/wifi/test/spectrum-wifi-phy-test.cc View 1 chunk +1 line, -1 line 0 comments Download
M src/wifi/test/wifi-test.cc View 1 2 3 4 5 3 chunks +189 lines, -0 lines 0 comments Download
A src/wifi/test/wifi-transmit-mask-test.cc View 1 2 1 chunk +716 lines, -0 lines 0 comments Download
M src/wifi/wscript View 1 2 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 23
Rediet
Hello Sebastien, Michael, Skalpa, everyone, This patch models the transmit spectrum mask defined for OFDM-based ...
3 months ago (2017-10-18 08:35:32 UTC) #1
S. Deronne
Rediet, thanks for your nice job. I am wondering: could you add unit tests? This ...
2 months, 3 weeks ago (2017-10-24 19:26:05 UTC) #2
S. Deronne
https://codereview.appspot.com/329500043/diff/1/src/wifi/model/spectrum-wifi-phy.cc File src/wifi/model/spectrum-wifi-phy.cc (right): https://codereview.appspot.com/329500043/diff/1/src/wifi/model/spectrum-wifi-phy.cc#newcode381 src/wifi/model/spectrum-wifi-phy.cc:381: guardBandwidth = uint32_t (GetChannelWidth ()); why this? In case ...
2 months, 3 weeks ago (2017-10-24 19:26:16 UTC) #3
S. Deronne
https://codereview.appspot.com/329500043/diff/1/src/wifi/examples/wifi-trans-gnuplot.plg File src/wifi/examples/wifi-trans-gnuplot.plg (right): https://codereview.appspot.com/329500043/diff/1/src/wifi/examples/wifi-trans-gnuplot.plg#newcode42 src/wifi/examples/wifi-trans-gnuplot.plg:42: #pause -1 Why do you need a separate file? ...
2 months, 3 weeks ago (2017-10-24 19:27:25 UTC) #4
micha.rademacher
Dear Rediet, Thanks a lot for the effort. Sadly, I am unable to get any ...
2 months, 3 weeks ago (2017-10-25 13:16:19 UTC) #5
micha.rademacher
Some c++ compile erros. https://codereview.appspot.com/329500043/diff/1/src/spectrum/model/wifi-spectrum-value-helper.cc File src/spectrum/model/wifi-spectrum-value-helper.cc (right): https://codereview.appspot.com/329500043/diff/1/src/spectrum/model/wifi-spectrum-value-helper.cc#newcode249 src/spectrum/model/wifi-spectrum-value-helper.cc:249: Values::iterator vit = c->ValuesBegin (); ...
2 months, 3 weeks ago (2017-10-25 13:16:47 UTC) #6
Rediet
On 2017/10/24 19:26:05, S. Deronne wrote: > Rediet, thanks for your nice job. > I ...
2 months, 2 weeks ago (2017-10-30 14:54:32 UTC) #7
Rediet
Thanks Sébastien, thanks Michael. https://codereview.appspot.com/329500043/diff/1/src/spectrum/model/wifi-spectrum-value-helper.cc File src/spectrum/model/wifi-spectrum-value-helper.cc (right): https://codereview.appspot.com/329500043/diff/1/src/spectrum/model/wifi-spectrum-value-helper.cc#newcode249 src/spectrum/model/wifi-spectrum-value-helper.cc:249: Values::iterator vit = c->ValuesBegin (); ...
2 months, 2 weeks ago (2017-10-30 15:07:57 UTC) #8
Rediet
On 2017/10/25 13:16:19, micha.rademacher wrote: > Dear Rediet, > > Thanks a lot for the ...
2 months, 2 weeks ago (2017-10-30 15:13:34 UTC) #9
Rediet
https://codereview.appspot.com/329500043/diff/1/src/wifi/examples/wifi-trans-gnuplot.plg File src/wifi/examples/wifi-trans-gnuplot.plg (right): https://codereview.appspot.com/329500043/diff/1/src/wifi/examples/wifi-trans-gnuplot.plg#newcode42 src/wifi/examples/wifi-trans-gnuplot.plg:42: #pause -1 On 2017/10/30 15:07:57, Rediet wrote: > On ...
2 months, 2 weeks ago (2017-10-30 16:21:42 UTC) #10
Rediet
On 2017/10/30 15:13:34, Rediet wrote: > On 2017/10/25 13:16:19, micha.rademacher wrote: > > Dear Rediet, ...
2 months, 2 weeks ago (2017-10-31 12:52:09 UTC) #11
Rediet
Dear Sébastien, Michael, Here is a second patch taking into account your comments/corrections, namely: - ...
2 months, 1 week ago (2017-11-07 13:05:52 UTC) #12
S. Deronne
Thanks, I still need to find some time to review the helper class itself. Can ...
2 months, 1 week ago (2017-11-07 19:08:22 UTC) #13
Rediet
Thanks Sébastien. My replies below. https://codereview.appspot.com/329500043/diff/20001/src/wifi/test/wifi-test-spectrum-value-h.cc File src/wifi/test/wifi-test-spectrum-value-h.cc (right): https://codereview.appspot.com/329500043/diff/20001/src/wifi/test/wifi-test-spectrum-value-h.cc#newcode30 src/wifi/test/wifi-test-spectrum-value-h.cc:30: NS_LOG_COMPONENT_DEFINE ("WifiTestSpectrumValueHelper"); On 2017/11/07 ...
2 months, 1 week ago (2017-11-08 10:51:41 UTC) #14
S. Deronne
https://codereview.appspot.com/329500043/diff/20001/src/spectrum/model/wifi-spectrum-value-helper.cc File src/spectrum/model/wifi-spectrum-value-helper.cc (right): https://codereview.appspot.com/329500043/diff/20001/src/spectrum/model/wifi-spectrum-value-helper.cc#newcode175 src/spectrum/model/wifi-spectrum-value-helper.cc:175: double lowestPointDbr = -40.0; this is not needed in ...
2 months, 1 week ago (2017-11-09 07:15:17 UTC) #15
Rediet
Dear Sébastien, Thanks for your comments (and sorry for the delay, just managed to find ...
1 month, 3 weeks ago (2017-11-22 16:22:39 UTC) #16
S. Deronne
https://codereview.appspot.com/329500043/diff/20001/src/spectrum/model/wifi-spectrum-value-helper.cc File src/spectrum/model/wifi-spectrum-value-helper.cc (right): https://codereview.appspot.com/329500043/diff/20001/src/spectrum/model/wifi-spectrum-value-helper.cc#newcode175 src/spectrum/model/wifi-spectrum-value-helper.cc:175: double lowestPointDbr = -40.0; I agree with those values, ...
1 month, 3 weeks ago (2017-11-22 16:53:20 UTC) #17
Rediet
Dear Sébastien, Thank you for your review of the second patch and sorry for the ...
1 month, 1 week ago (2017-12-07 15:07:47 UTC) #18
S. Deronne
Rediet, thanks for your new patchset, I have some small remarks, but I did not ...
1 month, 1 week ago (2017-12-09 16:24:11 UTC) #19
skalpaiit
Dear Rediet Thanks for patch and all the recent updates and modifications. Here are some ...
1 month ago (2017-12-15 08:36:11 UTC) #20
Rediet
Dear Sébastien, S.Kalpalatha, Nicola, Thanks for your comments Sébastien and S.Kalpalatha. Nicola could you please ...
1 month ago (2017-12-15 15:05:44 UTC) #21
Rediet
Dear Sébastien, I've updated the patch with bug's 2843's corrections. I think I've covered all ...
1 week, 4 days ago (2018-01-05 11:10:06 UTC) #22
Rediet
3 days, 21 hours ago (2018-01-12 14:31:35 UTC) #23
Updated version following Sebastien's recommendation in bug 2483.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 204d58d