search

openscd / open-scd

A substation configuration description editor for projects using SCL IEC 61850-6 Edition 2 or greater
https://openscd.github.io
Apache License 2.0
97 stars 31 forks source link

Automatic LNode removals and adding LNode interface #1315

Open danyill opened 1 year ago

danyill commented 1 year ago

Describe the bug

We expect to use LNode elements in the Substation section with references to the Protection 1 and Protection 2 systems.

We also expect to use separate files for the Protection 1 and Protection 2 system due to incompatibilities with ICTs.

Currently, OpenSCD silently and automatically removes LNode entries which do not have a valid iedName attribute.

Additionally when unreferenced LNodes exist, OpenSCD opens the new LNode screen at an incorrect initial starting point.

To Reproduce

Steps to reproduce the behavior:

The attached file is an extracted Substation section without some of the IEDs and their instantiated LNodes:

Prot1_Test_File_With_Substation.scd.zip

  1. Open the attached file.
  2. Go to the BusPhysConnection for one of the buses within the Substation editor.
  3. Click on the Add LNode menu item from the drop down menu: image
  4. Click the three dots and select, "Add reference to existing logical node" image
  5. Notice that the screen is already partway through the wizarding process (we have jumped the IED selection process). image
  6. Return to the IED selection screen.
  7. Select any IED and then any logical node and then select Save
  8. The existing logical nodes are removed: image

Expected behavior

  1. When a reference is added to an existing logical node the starting screen should be the IED selection screen.
  2. Invalid or incorrect references to IEDs in the LNode section should not be automatically removed. The SSD file may be managed without instantiated IEDs.

Screenshots

See above.

Additional Information

See

trusz commented 1 year ago

As discussed this seems to be two separate bugs:

JakobVogelsang commented 1 year ago

@danyill The file you have pushed does not have any Substation section. It is hard for me to reproduce.

danyill commented 12 months ago

Sorry @JakobVogelsang I attached an incorrect file. Please use the attached:

TPMulticastNaming-Test-2Bus-Prot2 (1).scd.zip

github-actions[bot] commented 10 months ago

Hello there,

Thank you for opening this issue! We appreciate your interest in our project. However, it seems that this issue hasn't had any activity for a while. To ensure that our issue tracker remains organized and efficient, we occasionally review and address stale issues.

If you believe this issue is still relevant and requires attention, please provide any additional context, updates, or details that might help us understand the problem better. Feel free to continue the conversation here.

If the issue is no longer relevant, you can simply close it. If you're uncertain, you can always reopen it later.

Remember, our project thrives on community contributions, and your input matters. We're here to collaborate and improve. Thank you for being part of this journey!