Skip to content

Commit

Permalink
tests: Fix after merging two incompatible changes
Browse files Browse the repository at this point in the history
Oops. We are not using gating, which means that changes are tested
against the "current tip of the branch" and might pass fine there, but
once they are merged, there can well be a conflict between them. This
has just happened.

The EDFA which reported a difference had its VOA set to 0.5. Previously,
this was not taken into assumption.

Fixes: ce51a4d Take explicitly set out_voa value into account in power calculation
Fixes: 280443f add an invocation test with power saturation
Change-Id: Icebbb16d2ef5886d2c9c04cc9a300a6aa08bf245
  • Loading branch information
jktjkt committed Sep 15, 2021
1 parent 0dc7d85 commit 6bb9ae8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/invocation/transmission_saturated
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ max flat gain: 16dB ; required gain: 23.0dB. Please check amplifier type.
a power reduction of -1.82 is applied

WARNING: WARNING: effective gain in Node east edfa in Lannion_CAS to Morlaix is above user specified amplifier std_low_gain
max flat gain: 16dB ; required gain: 23.0dB. Please check amplifier type.
max flat gain: 16dB ; required gain: 23.5dB. Please check amplifier type.
WARNING: target gain and power in node west edfa in Brest_KLA to Morlaix
is beyond all available amplifiers capabilities and/or extended_gain_range:
a power reduction of -1.82 is applied
Expand Down

0 comments on commit 6bb9ae8

Please sign in to comment.