-
Notifications
You must be signed in to change notification settings - Fork 31
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
feat(wizard/connectedap): create GSE and SMV elements #1019
Conversation
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.
Just a few little comments, generally very nice and a learning experience for me, gracias 👍
I had some fun testing this by completing removing all subnets in a large file, creating them and watching the SMV and GSE elements get created. LGTM.
I did not quite understand. What I do see is the same GSE is created with the same MAC-address in each different ConnectedAP that a device supports. So for instance if a device has three ConnectedAPs (say Is that what was intended? I might have guessed only creating in one subnetwork is what was required. |
Not at all. This is the behaviour I am after: Say you have 3 accP
I this the behavior you can see with your files? |
Thanks for the explanation, I am somewhat convinced. We will need ways to quickly allow users to select and remove if we don't provide this option while adding the ConnectedAP elements to the Subnet. In the attached, I have merged the two branches (read only and this GSE/SMV element creator), then removed all subnetworks. Then created one. Then I have added all ConnectedAP elements to this subnet. Quite a few of the IEDs have ServerAt directives and multiple AccessPoints. After this I extract and take a look at the Communication Section.
I think you intended for the MAC address and APPIDs to be globally unique. But we iterate based on the AccessPoint and reinstantiate the lists at the start. |
I got the issue. It happens when you select to create multiple |
6d79005
to
faa5554
Compare
Should be fixed now. Thanks for testing :) |
@danyill would you be so kind and have another look at this one :) |
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.
LGTM.
Closes #1013
This PR shall add
GSE
andSMV
elements on creating a newConnectedAP
. This is needed as a preparation for the subscribing plugins. The logic here isGSEControl
addGSE
element with uniqueMAC-Address
andAPPID
SampledValueControl
addSMV
element with uniqueMAC-Address
andAPPID
Server
element andServerAt
elementI did want to prevent UI components and with that have added two new generator functions to the foundation:
they can be re-used to generate unique MAC-Addresses and APPIDs within the valid ranges of both.