View Issue Details

IDProjectCategoryView StatusLast Update
0007667Part 81: UAFX Connecting Devices and Information Model [sg.BaseFacet]Specpublic2022-05-30 22:35
ReporterGeorg Biehler Assigned ToGeorg Biehler  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version1.00.00 RC2 
Target Version1.00.00 RC3Fixed in Version1.00.00 RC3 
Summary0007667: How to "add the PubSubConfiguration"
Description

Part 81 is not precise enough on the processing of the members in an instance of PubSubCommunicationConfigurationDataType. Currently it only states “shall add the supplied PubSubConfiguration to PubSub as described in OPC 10000-14”. It is not clear to a reader, that this refers to “9.1.3.7 Modification of PubSub configuration” in part 14.

It is a far stretch, that a reader will look up the datatype, see that it is defined in Part 14, used in a method that is called CloseAndUpdate, and then deduces that this is what is meant in Part 81. That the arguments have matching names may help, but this is really “educated guessing” from the reader’s side and in my opinion a specification should not rely on readers guessing correctly.

Also: An implementer of EstablishConnections, in order to reuse a potentially existing CloseAndUpdate function, must write the configuration to a file, open the file, and pass a file handle to the SDK function.
The file handle is an Uint32 and in part 14 it is probably returned by the FileType.Open() method, which will in turn require the client of CloseAndUpdate to create a FileType object first.
It does seem to me, that EstablishConnections will not be able to use an existing SDK function without any coding effort (which was an initial assumption I believe).

The description of the ConfigurationReferences member in Part 81 mentions a file. Part 81 does not provide a possibility yet to upload a file, thus a reader will not know what this refers to.

(copied from a Mail Thread between David, Brian and Georg)

TagsNo tags attached.

Activities

Paul Hunkar

2022-02-01 14:26

manager   ~0015899

Discussed issue in working group.
The topic is not discussed on how to implement the configuration into the PubSub, and although that was intention in the original spec (i.e. left to vendor). This feedback is that maybe that to open, and that some level of guidance should be provided. It may not be that we want to prescribe how to implement this, but that some suggestion on how it could be mapped to the part 14 specification should be made and certain topics such as matching an existing configuration should be discussed in more details.

Georg Biehler

2022-02-23 13:56

developer   ~0016080

We need to reference also to the StatusCode description in Part14 UpdatAndClose, which is expected independent how the SetCommunicationCmd is implemented.

Paul Hunkar

2022-02-24 03:49

manager   ~0016083

add a file point to the union for the pubsubconfiguration2datatype

other items that should be discussed (pubsub persistence - opposite of FX persistence

Georg Biehler

2022-02-28 09:25

developer   ~0016118

Last edited: 2022-02-28 10:15

Added FxPubSubConfiguration - a union allowing file and data
Added text to EstablishConnections for guidance how to map (Part14 CloseAndUpdate, or vendor specific implementation)
Added text to ConnectionManager for handling the file
Added text to PubSubCommunicationModelConfigurationType for use of the operation flags

Georg Biehler

2022-02-28 10:25

developer   ~0016119

Updated spec

Paul Hunkar

2022-04-11 13:35

manager   ~0016563

Reviewed changes in call, agreed to text and closed issue

Issue History

Date Modified Username Field Change
2022-01-27 10:13 Georg Biehler New Issue
2022-02-01 14:26 Paul Hunkar Note Added: 0015899
2022-02-01 14:26 Paul Hunkar Assigned To => Paul Hunkar
2022-02-01 14:26 Paul Hunkar Status new => assigned
2022-02-23 13:56 Georg Biehler Note Added: 0016080
2022-02-24 03:49 Paul Hunkar Note Added: 0016083
2022-02-24 13:41 Georg Biehler Assigned To Paul Hunkar => Georg Biehler
2022-02-28 09:25 Georg Biehler Note Added: 0016118
2022-02-28 10:15 Georg Biehler Note Edited: 0016118
2022-02-28 10:25 Georg Biehler Status assigned => resolved
2022-02-28 10:25 Georg Biehler Resolution open => fixed
2022-02-28 10:25 Georg Biehler Fixed in Version => 1.00.00 Release
2022-02-28 10:25 Georg Biehler Note Added: 0016119
2022-04-11 13:35 Paul Hunkar Status resolved => closed
2022-04-11 13:35 Paul Hunkar Note Added: 0016563
2022-05-30 22:35 Paul Hunkar Fixed in Version 1.00.00 Release => 1.00.00 RC3
2022-05-30 22:35 Paul Hunkar Target Version 1.00.00 Release => 1.00.00 RC3