View Issue Details

IDProjectCategoryView StatusLast Update
0007734Part 81: UAFX Connecting Devices and Information ModelSpecpublic2024-08-16 13:10
ReporterJan Murzyn Assigned ToPaul Hunkar  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionno change required 
Summary0007734: Occurences of PortableNodeIdentifier need revision of RelativePaths roots.
Description
  • Description of the root of the relative path is missing at some places.

  • RelativePaths can be optimized in several places by skipping initial path elements which are know anyway. That will make a significant difference for the total size of configuration, because the number of identifiers included in configuration can be huge.

6.11.2 ConnectionEndpointConfigurationType - FunctionalEntityNode
Currently description says the path is relative to FxRoot. To shorten the path, it can be made relative to the AC. The containing AC is known thanks to the ToConnectionEndpointConfiguration reference.

6.12.2 ConnectionEndpointDefinitionType - Node
Root not specified. Relative to FE?

6.11.2 ConnectionEndpointConfigurationType - ExpectedVerificationVariables, ControlGroups, ConfigurationData
Currently description says the path is relative to FunctionalEntityNode. But it can be relative to the corresponding folder within the FE.

6.13.2 ConnectionEndpointParameterType definition - InputVariableIds, InputGroupId, OutputVariableIds, OutputGroupId
For groups, the root of the path has to be FE (to account for the fact that the top level OutputData and InputData could be specified). But for individual variables, start of the path can be the corresponding Input or Output folder, not FE as it is at the moment.

6.16.2 AssetVerificationType definition - AssetToVerify
Can be relative to Assets folder in AC.

6.16.2 AssetVerificationType definition - ExpectedAdditionalVerificationVariables
Root not specified. Relative to what?

TagsNo tags attached.

Relationships

related to 0007863 closedPaul Hunkar Missing starting node for browsePath 
related to 0007864 closedGeorg Biehler ConnectionEndpointDefinitionType - Node has wrong type 

Activities

Paul Hunkar

2022-02-15 13:14

manager   ~0016008

Last edited: 2022-04-19 15:31

-A general comment - the relative path is built in the engineering tool - which only have the AC NodeId or the well known FXRoot item in many cases - the nodeid of a folder is not required for the descriptor - so it can not be a starting point from an engineering tool point of view.

6.11.2 ConnectionEndpointConfigurationType - FunctionalEntityNode
Currently description says the path is relative to FxRoot. To shorten the path, it can be made relative to the AC. The containing AC is known thanks to the ToConnectionEndpointConfiguration reference.

Paul - The ToConnectionEndpointConfiguration is a non-hierarchical reference, which is not easy to follow (translate Browse paths does not process it, the FLCRoot is a well known object and can be hardcoded in the path in the engineering tool

6.12.2 ConnectionEndpointDefinitionType - Node
Root not specified. Relative to FE?

paul - the mistake here is that this should be a PortableNodeid not an identifier - this should be fixed (it is where the Nodeid is stored once parameter is processed)

6.11.2 ConnectionEndpointConfigurationType - ExpectedVerificationVariables, ControlGroups, ConfigurationData
Currently description says the path is relative to FunctionalEntityNode. But it can be relative to the corresponding folder within the FE.

paul - for verification we have said it can be anything in the FE - so this is correct, , for controlgroup - it can be control groups in sub functionalentities - so again it is correct, same a control groups - can be sub functionalentities

6.13.2 ConnectionEndpointParameterType definition - InputVariableIds, InputGroupId, OutputVariableIds, OutputGroupId
For groups, the root of the path has to be FE (to account for the fact that the top level OutputData and InputData could be specified). But for individual variables, start of the path can be the corresponding Input or Output folder, not FE as it is at the moment.

paul - again nested functionalentities require the FE, instead of the folder.

6.16.2 AssetVerificationType definition - AssetToVerify
Can be relative to Assets folder in AC.

paul - agreed that this could be, but the assetfolder would have to first be found - since that nodeid is not know in the descriptor (by the engineering tool) - at least I don;t think it is in the descriptor- it would reequire a two step processing - I think it is easier to just start at the AC - which is already being provided.

6.16.2 AssetVerificationType definition - ExpectedAdditionalVerificationVariables
Root not specified. Relative to what?

paul - Agreed this one is missing the starting definition, but I'm not sure we have the type correct since the variable has to be defined in the Asset (if you look at the method) - I think a relative path is incorrect this should be protableNodeis, or PortableBrowsename, nothing else.

Paul Hunkar

2022-02-24 16:43

manager   ~0016094

In call determined that there might be a manner to add a generic path to CM so the configuration in the configuration set might not need the additional path items. This possible option will be illustrated to all via a PPT, before this issue is assigned.

Paul Hunkar

2022-03-15 14:21

manager   ~0016379

Last edited: 2022-03-15 15:22

Discuss in call - that this could be handled in vendor specific implementation (think string pools), issue can be acknowledge and left for next release if it show to be an issue - otherwise can be close in next release.

Attached PPT that include all on the initial discussion on the issue

Paul Hunkar

2022-03-15 15:21

manager   ~0016386

BrowsePath.pptx (647,473 bytes)

Paul Hunkar

2022-11-04 13:29

manager   ~0018110

What is remaining at this point is a possible optimization - waiting for implementations to determine if anything is actually needed

Paul Hunkar

2024-07-11 17:56

manager   ~0021450

All Agree nothing more to do

Paul Hunkar

2024-08-16 13:10

manager   ~0021574

agreed in prototyping that nothing more is needed

Issue History

Date Modified Username Field Change
2022-02-15 10:26 Jan Murzyn New Issue
2022-02-15 13:14 Paul Hunkar Note Added: 0016008
2022-02-24 16:43 Paul Hunkar Note Added: 0016094
2022-03-15 14:21 Paul Hunkar Note Added: 0016379
2022-03-15 15:21 Paul Hunkar Note Added: 0016386
2022-03-15 15:21 Paul Hunkar File Added: BrowsePath.pptx
2022-03-15 15:22 Paul Hunkar Status new => acknowledged
2022-03-15 15:22 Paul Hunkar Note Edited: 0016379
2022-03-15 15:25 Paul Hunkar Relationship added related to 0007863
2022-03-15 15:36 Paul Hunkar Relationship added related to 0007864
2022-04-19 15:31 Paul Hunkar Note Edited: 0016008
2022-04-19 15:32 Paul Hunkar Severity major => minor
2022-11-04 13:29 Paul Hunkar Note Added: 0018110
2024-07-11 17:56 Paul Hunkar Assigned To => Paul Hunkar
2024-07-11 17:56 Paul Hunkar Status acknowledged => resolved
2024-07-11 17:56 Paul Hunkar Resolution open => no change required
2024-07-11 17:56 Paul Hunkar Note Added: 0021450
2024-08-16 13:10 Paul Hunkar Status resolved => closed
2024-08-16 13:10 Paul Hunkar Note Added: 0021574