HDAIS adds to the DAIS module. This opens for a potential name clashes.
Discussion:
Investigate if to keep the current module structure or inclusion by redefinition. Make HDAIS functionality including DAIS. Move the HDA server up to the DAIS namespace.

see above

There is no interface a client can use to get the level of conformance actually supported by a server.
Discussion:
Standardize the conformance levels and make them available through an interface at Server. Also do this for DAIS DA and A&E.

Add one constant for server functions DA, A&E and HDA as well as all the HDA functions

The find_each call does not allow reporting on individual UnkonwItemIDs. The find_by calls does not support exceptions for all parameters, e.g. UnkownTypeIDs.
Discussion:
Update the exceptions to be consistent.

Change as suggested

It is unclear if the CancelIDs need to be globally unique.
Discussion:
The CancelIDs are server assigned and must be globally unique. Make this clear in the text. Figure 5 does not show the session-callback object cardinality clearly. Fix this.

Change made as suggested

The File HDAISItemAttribute.idl that I retrieved from
http://www.omg.org/cgi-bin/doc?mantis/02-10-04
contains several errors:

The parameter name is missing on line 66 and 70,
and the type AbsoluteIntervals (lines 41 and 51)
does not exist.

I assume that TimeInterval is the correct type.

rejected

The text discussing bridging to OPC gives the impression this is difficult.
Discussion:
Make the wording sound more positive.

Change the text

Why are some source lines commented out and not removed?
Discussion:
Remove commented source lines.

The commented lines are kept and the comment slashes removed

It is difficult to understand the conformance table in section 1.6.
Discussion:
Give the conformance points clear names so that they can be easily identified. Add a list with the named conformance points after the table in section 1.6.

No Data Available

find_by_parent() and find_by_typ shall take multiple filter spec rather than a single, i.e. use AttributeFilters instead of AttributeFilter.

Change as described

WARNING_MORE_EXTREEM_VALUES wrongly spelled, should be WARNING_MORE_EXTREME_VALUES.

Change all WARNING_MORE_EXTREEM_VALUES to WARNING_MORE_EXTREME_VALUES

The OPC HDA AGGREGATE defined for numbers 0x0e to 0x13 are missing for AggregateID. This complicates bridges. Change the AggregateID numbering to include the OPC HDA AGGREGATE numbers 0x0e to 0x13

The numbers was erroneously defined and shall be corrected

Is it necessary to add the PropertyID to SimpleValue?
Discussion:
If this can be avoided there is no need to change the DAF spec.

No, SimpleValue will not be extended with PropertyID.

The singular and plural forms are the same for time series, i.e. there is no English word time serie. Replace the word TimeSerie with TimeSequence or TimeSeriesItem.

rejected

The OPC HDA AGGREGATE defined for numbers 0 to 0x7FFFFFFF are reserved by OPC foundation. They shall be reserved for HDAIS as well. Add text making this reservation.

Change as described

OPC has added a new quality code bit OPCHDA_PARTIAL in HDA 1.20. Add also to HDAIS

Add the code

The AttributeFilter "operator" clash with the C++ keyword operator. Change to "opc_operator".

Change as suggested

The cancel method is lacking on the async Playback and AsyncUpdate interfaces. Either add the methods or move it to the Session object

Have one cancel method on the Connection (Session) object used for all cancelations

SimpleValue members in different structs are named differently, e.g. val, simple_value, the_value. Use a consistent naming, e.g. simple_value.

Change as described

In DAIS there is an attribute holding the callback object. The same solution shall be used in HDAIS.

Add callback.