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.

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

Booster amplifier and Pre-amplifier for the 'rdm' nodetype #77

Open
abhiman84 opened this issue Jul 10, 2024 · 1 comment
Open

Booster amplifier and Pre-amplifier for the 'rdm' nodetype #77

abhiman84 opened this issue Jul 10, 2024 · 1 comment

Comments

@abhiman84
Copy link

Please refer to line numbers 2514 and 2515 in org-openroadm-device.yang.

list line-amplifier { when "/org-openroadm-device/info/node-type='ila'";

Here line-amplifier is allowed only for 'ila' node and not 'rdm'. However we would like to have booster amplifier and pre-amplifier which have the same structures as amplifier.

The 'target-tilt' and 'target-gain' of pre-amplifier and booster amplifier should be configurable for our use case. I would be grateful if I can get some leads to achieve this.

@fgruman
Copy link

fgruman commented Oct 16, 2024

@abhiman84
A history of the model development was that the ROADM models were created first, and then later the ILA models were added.

At the time when the ROADM was being modeled, it was decided that Open ROADM would support a black box view for the ROADM. The API would support the ability to set the target output power per service (this attribute appears in the roadm-connection object). The device would then determine the internal amplifier gains and tilt to achieve the channel power settings.

Please also note that at this time Open ROADM has been assuming a flat power launch model (actually constant PSD) and have not done much with tilt or pre-emphasis.

When ILAs were developed, the ILA control was agreed to be to allow for the control of the target gain and tilt. There was a debate within the forum if these should apply back to the ROADM node but it was decided to not do this and continue with the black box behavior for the ROADM. This is the reason you see the when statement you mention above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants