Bug 1043 - Classes static_LteDownlinkSpectrumModel_initializer and static_LteUplinkSpectrumModel_initializer have swapped the operating frequency band
Classes static_LteDownlinkSpectrumModel_initializer and static_LteUplinkSpect...
Status: RESOLVED FIXED
Product: ns-3
Classification: Unclassified
Component: lte
ns-3-dev
All All
: P5 normal
Assigned To: Giuseppe Piro
: bug, patch
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-01-14 13:33 UTC by ritwik
Modified: 2011-08-16 16:04 UTC (History)
2 users (show)

See Also:


Attachments
This diff file will swap the uplink and downlink freq band usage by the two classes in lte-spectrum-value-helper.cc file (1.12 KB, patch)
2011-01-14 13:33 UTC, ritwik
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description ritwik 2011-01-14 13:33:59 UTC
Created attachment 1024 [details]
This diff file will swap the uplink and downlink freq band usage by the two classes in lte-spectrum-value-helper.cc file

The file src/devices/lte/model/lte-spectrum-value-helper.cc have implemented classes static_LteDownlinkSpectrumModel_initializer and static_LteUplinkSpectrumModel_initializer. The downlink operating frequency band for LTE is 2110 MHz – 2170 MHz and the uplink operating frequency band is 1920 MHz – 1980 MHz.


Actual Code: 
The constructors of these classes have swapped the downlink and uplink frequency bands. 

static_LteDownlinkSpectrumModel_initializer::static_LteDownlinkSpectrumModel_initializer() is using 1920 MHz – 1980 MHz band

static_LteUplinkSpectrumModel_initializer::static_LteUplinkSpectrumModel_initializer() is using 2110 MHz – 2170 MHz band.

Expected Code:
The frequency band usage should be swapped. I've attached a diff file to correct the code file.

Version info:
Found in ns-3.10 and ns-3-dev
Comment 1 Giuseppe Piro 2011-03-24 15:29:43 UTC
Fixed into the 
changeset 6883     d2770ff543f7
of ns-3-lte
Comment 2 Tom Henderson 2011-08-09 15:30:38 UTC
(In reply to comment #1)
> Fixed into the 
> changeset 6883     d2770ff543f7
> of ns-3-lte

can you fix to ns-3-dev and close this, so that it is in ns-3.12 (since ns-3-lte won't be merged until after ns-3.12)?
Comment 3 Tom Henderson 2011-08-16 16:04:39 UTC
fix pulled from Giuseppe's repo:  changeset ca0281bce8a9