Skip to content

Commit bfe5540

Browse files
Fix for modulation display and power strings
Change-Id: I6d74b6d2d0946c8fac456f5a9e8e65571b9b1136
1 parent a5c80c6 commit bfe5540

File tree

3 files changed

+10
-14
lines changed

3 files changed

+10
-14
lines changed

apps/roadm/app/src/main/java/org/onosproject/roadm/RoadmPortViewMessageHandler.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -256,7 +256,7 @@ private String getModulation(DeviceId deviceId, PortNumber portNumber) {
256256
if (port.type().equals(Port.Type.OCH)) {
257257
modulation = roadmService.getModulation(deviceId, portNumber);
258258
}
259-
return RoadmUtil.objectToString(modulation, RoadmUtil.UNKNOWN);
259+
return RoadmUtil.objectToString(modulation, RoadmUtil.UNKNOWN).toLowerCase();
260260
}
261261

262262
// Returns modulation as a string, Unknown if modulation is expected but

apps/roadm/app/src/main/java/org/onosproject/roadm/RoadmUtil.java

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -131,15 +131,15 @@ public static OchSignal createOchSignalFromWavelength(double wavelength, DeviceS
131131
GridType gridType = ochPort.lambda().gridType();
132132
ChannelSpacing channelSpacing = ochPort.lambda().channelSpacing();
133133
int slotGranularity = ochPort.lambda().slotGranularity();
134-
int multiplier = getMultplier(wavelength, gridType, channelSpacing);
134+
int multiplier = getMultiplier(wavelength, gridType, channelSpacing);
135135
return new OchSignal(gridType, channelSpacing, multiplier, slotGranularity);
136136
} else {
137137
return null;
138138
}
139139

140140
}
141141

142-
private static int getMultplier(double wavelength, GridType gridType, ChannelSpacing channelSpacing) {
142+
private static int getMultiplier(double wavelength, GridType gridType, ChannelSpacing channelSpacing) {
143143
long baseFreq;
144144
switch (gridType) {
145145
case DWDM:
@@ -152,10 +152,6 @@ private static int getMultplier(double wavelength, GridType gridType, ChannelSpa
152152
baseFreq = 0L;
153153
break;
154154
}
155-
if (wavelength > baseFreq) {
156-
return (int) ((wavelength - baseFreq) / (channelSpacing.frequency().asMHz()));
157-
} else {
158-
return (int) ((baseFreq - wavelength) / (channelSpacing.frequency().asMHz()));
159-
}
155+
return (int) ((wavelength - baseFreq) / (channelSpacing.frequency().asMHz()));
160156
}
161157
}

apps/roadm/web/roadm-gui/projects/roadm-gui-lib/src/lib/port/port.component.html

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -73,9 +73,9 @@ <h2>
7373
<td colId="currentFreq">CURRENT FREQ (THz)</td>
7474
<td colId="modulation">MODULATION</td>
7575
<td colId="powerRange">POWER RANGE (dBm)</td>
76-
<td colId="currentPower">CURRENT POWER (dBm)</td>
76+
<td colId="currentPower">CURRENT OUTPUT POWER (dBm)</td>
7777
<td colId="currentInputPower">CURRENT INPUT POWER (dBm)</td>
78-
<td colId="targetPower">TARGET POWER (dBm)</td>
78+
<td colId="targetPower">TARGET OUTPUT POWER (dBm)</td>
7979
<td colId="hasTargetPower">HAS TARGET POWER</td>
8080
<td colId="serviceState">SERVICE STATE</td>
8181
</tr>
@@ -94,9 +94,9 @@ <h2>
9494
<td [ngClass]="(isDelta() ? 'delta' : '')">{{port.name}}</td>
9595
<td [ngClass]="(isDelta() ? 'delta' : '')">{{port.type}}</td>
9696
<td [ngClass]="(isDelta() ? 'delta' : '')">{{port.enabled}}</td>
97-
<td [ngClass]="(isDelta() ? 'delta' : '')">{{port.type=='OCH'?port.minFreq:""}}</td>
98-
<td [ngClass]="(isDelta() ? 'delta' : '')">{{port.type=='OCH'?port.maxFreq:""}}</td>
99-
<td [ngClass]="(isDelta() ? 'delta' : '')">{{port.type=='OCH'?port.grid:""}}</td>
97+
<td [ngClass]="(isDelta() ? 'delta' : '')">{{port.type=='OCH' || port.type=='OMS'?port.minFreq:""}}</td>
98+
<td [ngClass]="(isDelta() ? 'delta' : '')">{{port.type=='OCH' || port.type=='OMS'?port.maxFreq:""}}</td>
99+
<td [ngClass]="(isDelta() ? 'delta' : '')">{{port.type=='OCH' || port.type=='OMS'?port.grid:""}}</td>
100100
<td [ngClass]="(isDelta() ? 'delta' : '')">
101101
<form [formGroup]="freqForm" (ngSubmit)="submitFrequency(devId, port.id)" *ngIf="port.type=='OCH'">
102102
<input type="text" style="width:50px" value="{{port.type=='OCH'?port.currFreq:''}}" formControlName="newFreq" required>
@@ -116,7 +116,7 @@ <h2>
116116
<td [ngClass]="(isDelta() ? 'delta' : '')">{{convertNumber(port.currentPower) | number: '1.2-2'}}</td>
117117
<td [ngClass]="(isDelta() ? 'delta' : '')">{{convertNumber(port.currentInputPower) | number: '1.2-2'}}</td>
118118
<td [ngClass]="(isDelta() ? 'delta' : '')">
119-
<form [formGroup]="powerForm" (ngSubmit)="submitPower(devId, port.id)" *ngIf="port.type=='OCH'">
119+
<form [formGroup]="powerForm" (ngSubmit)="submitPower(devId, port.id)" *ngIf="port.type=='OCH' || port.type=='OMS'">
120120
<input type="number" step="any" style="width:50px" value="{{convertNumber(port.targetPower) | number: '1.2-2'}}" formControlName="newPower" required>
121121
<button type="submit">Submit</button>
122122
</form>

0 commit comments

Comments
 (0)