Bug 944 - change default ErrorRateModel from YansErrorRateModel to NistErrorRateModel
change default ErrorRateModel from YansErrorRateModel to NistErrorRateModel
Status: RESOLVED FIXED
Product: ns-3
Classification: Unclassified
Component: wifi
pre-release
All All
: P5 normal
Assigned To: Tom Henderson
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-06-15 14:42 EDT by Tom Henderson
Modified: 2010-07-20 10:35 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Henderson 2010-06-15 14:42:55 EDT
I'm proposing to change the ns-3 default for Wifi to use the new NistErrorRateModel, which is validated against testbed results:
http://www.nsnam.org/~pei/80211ofdm.pdf

The patch would be:
--- a/src/helper/yans-wifi-helper.cc	Tue Jun 15 11:27:51 2010 -0700
+++ b/src/helper/yans-wifi-helper.cc	Tue Jun 15 11:37:55 2010 -0700
@@ -183,7 +183,7 @@
 YansWifiPhyHelper::Default (void)
 {
   YansWifiPhyHelper helper;
-  helper.SetErrorRateModel ("ns3::YansErrorRateModel");
+  helper.SetErrorRateModel ("ns3::NistErrorRateModel");
   return helper;
 }

Note that this patch causes these four regression tests to fail, presumably because the Yans model is more optimistic, but this must be confirmed:
FAIL: TestSuite devices-mesh-flame-regression
FAIL: TestSuite devices-mesh-dot11s-regression
FAIL: TestSuite routing-aodv-regression
FAIL: TestSuite routing-olsr-regression

Another option to shift usage to this error rate model would be to keep YansWifiHelper untouched, and create a new NistWifiHelper that reused Yans components, and deprecate the usage of YansWifiHelper in the examples/tutorials.  Since "Nist" is limited to the error rate model, it seemed that the above patch might be more straightforward, however.

Any other opinions?
Comment 1 Nicola Baldo 2010-06-21 20:05:36 EDT
I agree that the best option is just to change the default error rate model and update the regression traces.
Comment 2 Tom Henderson 2010-06-24 01:02:55 EDT
(In reply to comment #1)
> I agree that the best option is just to change the default error rate model and
> update the regression traces.

I will take care of this after other feature merge issues are handled.
Comment 3 Tom Henderson 2010-07-20 10:35:56 EDT
changeset:  d5cfe99001f0

See http://www.nsnam.org/~pei/80211ofdm.pdf for technical details.