-
Notifications
You must be signed in to change notification settings - Fork 10
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.
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
I-D text update resolving #150, #151, and #157 #152
Conversation
This I-D update resolves: - #150 - #151 + pure editorial updates/corrections Co-Authored-By: Esther Le Rouzic <EstherLerouzic@users.noreply.github.com> Co-Authored-By: Gabriele Galimberti <ggalimbe56@gmail.com> Co-Authored-By: sergio belotti <sergio.belotti@nokia.com> Co-Authored-By: italobusi <12461741+italobusi@users.noreply.github.com>
This I-D update addesses: - #144 : some comments resolved - #153 : new DGE sub-section added - #154 : latest YANG code and tree view incorporated + pure editorial updates/corrections Co-Authored-By: Esther Le Rouzic <EstherLerouzic@users.noreply.github.com> Co-Authored-By: sergio belotti <sergio.belotti@nokia.com> Co-Authored-By: italobusi <12461741+italobusi@users.noreply.github.com> Co-Authored-By: Gabriele Galimberti <ggalimbe56@gmail.com>
This I-D update resolves: - #157 : Examples of how vendors can define their operational modes. + pure editorial updates/corrections Co-Authored-By: Esther Le Rouzic <EstherLerouzic@users.noreply.github.com> Co-Authored-By: sergio belotti <sergio.belotti@nokia.com> Co-Authored-By: italobusi <12461741+italobusi@users.noreply.github.com> Co-Authored-By: Gabriele Galimberti <ggalimbe56@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello, I think that the type of delta_power should not be gain
@@ -2886,7 +2941,7 @@ module: ietf-optical-impairment-topology | |||
| +--ro otsi-ref* [] | |||
| | +--ro otsi-carrier-ref? leafref | |||
| | +--ro e2e-mc-path-ref* leafref | |||
| +--ro delta-power? l0-types:power-in-dbm-or-null | |||
| +--ro delta-power? l0-types:gain-in-db-or-null |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
delta-power contains a ratio in dB. one may call it gain, however gain is only positive, where delta_p could be negative. so the type should be changed
| +--ro flexi-m? | ||
| | l0-types:flexi-m | ||
| +--ro delta-power? | ||
| l0-types:gain-in-db-or-\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same comment
@@ -3823,7 +3937,7 @@ module ietf-optical-impairment-topology { | |||
} | |||
} | |||
leaf delta-power { | |||
type l0-types:power-in-dbm-or-null; | |||
type l0-types:gain-in-db-or-null; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same comment
This I-D update resolves: