Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Topology model extensions to support C+L band DWDM networks including all network topology components #51

Closed
dieterbeller opened this issue Feb 2, 2021 · 4 comments · Fixed by #68
Closed

Topology model extensions to support C+L band DWDM networks including all network topology components #51

dieterbeller opened this issue Feb 2, 2021 · 4 comments · Fixed by #68
Labels
enhancement New feature or request

Comments

@dieterbeller
Copy link
Member

dieterbeller commented Feb 2, 2021 •
edited
Loading

Following the optical amplifier model extensions to support C+L band amplifiers, model extensions shall be added for all the other network components in the topology model where L-band extensions are needed (e.g. ROADM model extensions).

A similar approach shall be used as for the amplifier model extensions where a copy of the C-band tree was added as a new sub-tree for the L-band to a new list containing these sub-trees for the different bands (See #46).
@dieterbeller dieterbeller added the enhancement New feature or request label Feb 2, 2021
@ggrammel
Copy link
Collaborator

ggrammel commented Feb 2, 2021

I can't find a "C-band tree" or "L-band tree" in #46 (comment).

@EstherLerouzic
Copy link
Collaborator

C+L booster or preamp amplifiers may have multiple input/output port from/to ROADM ports which makes the mapping of the OMS to the link more difficult. In addition to that, the current model might not be able to represent the cases when there is one more amp on one band (ILA or booster/preamp). I have added a drwing to explain this

@EstherLerouzic
Copy link
Collaborator

C+Lroadm.pptx

@EstherLerouzic
Copy link
Collaborator

Here is an update of the drawing: I think that the model can handle the particular cases... only the mapping issue remains... let's discuss this !
C+Lroadm_v2.pptx

sergiobelotti added a commit that referenced this issue Mar 2, 2021
@sergiobelotti
Create minutes-2021-03-01.md
e9d0a21 
The minute is related to a call related to issue #51 and comment from Esther #51 (comment)
italobusi added a commit that referenced this issue Apr 8, 2021
@italobusi
ROADM impairments for C+L band
6e9d2af 
Model updated to describe ROADM impairments for C+L band: resolving issue #51

Addressing also issue #64
@italobusi italobusi mentioned this issue Apr 8, 2021
Merged
@italobusi italobusi linked a pull request Apr 8, 2021 that will close this issue
ROADM impairments for C+L band #68
Merged
italobusi added a commit that referenced this issue Jun 15, 2021
@italobusi @sergiobelotti
ROADM impairments for C+L band (#68)
e7990fa 
Model updated to describe ROADM impairments for C+L band: resolving issue #51

Resolve issue #64

Co-authored-by: sergiobelotti <sergio.belotti@nokia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ROADM impairments for C+L band ietf-ccamp-wg/draft-ietf-ccamp-optical-impairment-topology-yang
3 participants
@EstherLerouzic @ggrammel @dieterbeller