View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006971 | 10000-003: Address Space | Spec | public | 2021-05-28 11:23 | 2025-08-22 13:57 |
| Reporter | Zbynek Zahradnik | Assigned To | Jim Luth | ||
| Priority | normal | Severity | major | Reproducibility | always |
| Status | assigned | Resolution | fixed | ||
| Fixed in Version | 1.05.00 | ||||
| Summary | 0006971: ValueRank should be considered together with DataType for NodeVersion changes | ||||
| Description | In Table 13 (Variable NodeClass) and Table 14 (VariableType NodeClass), we have following text with the NodeVersion property: "... Attribute value changes except for the DataType Attribute do not cause the NodeVersion to change." I think this should be extended to include the ValueRank attribute (together with DataType) as well, because the final type of the Value is given by a combination of DataType+ValueRank. At least for all my client-side usages, the ValueRank is significant. In fact, it is often more important than the data type itself; for example, a change from Integer to Byte isn't that important as a change from Integer to Integer[] (array of integers). | ||||
| Tags | No tags attached. | ||||
| Commit Version | |||||
| Fix Due Date | |||||
| related to | 0007344 | closed | Matthias Damm | 10000-004: Services | 7.34.1 StructureChange description not sufficient |
| related to | 0007030 | assigned | Jeff Harding | 10000-005: Information Model | ValueRank should be considered together with DataType for NodeVersion changes |
| related to | 0007340 | assigned | Jeff Harding | 10000-003: Address Space | Clarification needed for Server expectations regarding NodeVersion Properties |
|
|
If this change is accepted, then also Part 5 should change: In ModelChangeStructureDataType Structure, field 'verb' (Table 156 in 1.04), there is currently Bit 4 for DataTypeChanged: "This verb may be used only for affected Nodes that |
|
|
I agree with the logic that ValueRank should be considered in the same way DataType is. |
|
|
I agree - clients should be notified when any aspect of a Value changes - these can (and do) impact client behavior. A change in any attribute that changes the "shape" of the data should trigger this event. This includes ValueRank, ArrayDimensions, and DataType; perhaps others, such as access rights, whether or not writes w/timestamps are supported, etc. Further, when any of these attributes change, all related attributes should be updated to the extent their new value can be ascertained. For example,
This feature should be supported by a CU or facet that verifies this, |
|
|
We agreed in the June 15 2021 weekly call that a change to ValueRank and ArrayDimensions should also be considered in the same way as a DataType change. Need an errata for 1.04. |
|
|
Added ValueRank and ArrayDimensions in addition to DataType in Table 13 'Variable NodeClass' and Table 14 'VariableType NodeClass' Added Errata 1.04.11 |
|
|
Need to look at the bigger picture before committing to this solution. |
|
|
I reviewed the recoding from 2021-10-05 to understand why this change was reverted. There is a lot of disagreement about what does today, or should in the future trigger a ModelChangeEvent, a StructureChanged bit or a SemanticsChanged bit. On one hand, some changes seem to be missing that should be included, on the other hand things are included that are technically unnecessary as the change is always evident in every value change received (e.g. the dimensions of the array is always included with the value). For the particular case of ValueRank and ArrayDimensions -- see that no-fix was agreed to for Mantis 0007344 |
|
|
I added this to the Genoa F2F. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-05-28 11:23 | Zbynek Zahradnik | New Issue | |
| 2021-05-28 13:54 | Zbynek Zahradnik | Note Added: 0014438 | |
| 2021-06-14 14:27 | Jeff Harding | Note Added: 0014555 | |
| 2021-06-15 14:11 | David Levine | Note Added: 0014557 | |
| 2021-06-15 15:16 | Jeff Harding | Note Added: 0014559 | |
| 2021-06-15 15:21 | Jeff Harding | Issue cloned: 0007030 | |
| 2021-06-15 15:21 | Jeff Harding | Relationship added | related to 0007030 |
| 2021-06-15 15:24 | Jeff Harding | Assigned To | => Jeff Harding |
| 2021-06-15 15:24 | Jeff Harding | Status | new => assigned |
| 2021-09-27 18:49 | Jeff Harding | Status | assigned => resolved |
| 2021-09-27 18:49 | Jeff Harding | Resolution | open => fixed |
| 2021-09-27 18:49 | Jeff Harding | Fixed in Version | => 1.05.00 |
| 2021-09-27 18:49 | Jeff Harding | Note Added: 0015032 | |
| 2021-10-04 18:26 | Jeff Harding | Relationship added | related to 0007340 |
| 2021-10-05 16:43 | Jim Luth | Status | resolved => assigned |
| 2021-10-05 16:43 | Jim Luth | Note Added: 0015126 | |
| 2025-07-29 16:48 | Jim Luth | Note Added: 0023173 | |
| 2025-07-29 16:48 | Jim Luth | Assigned To | Jeff Harding => Jim Luth |
| 2025-07-29 19:56 | Jim Luth | Relationship added | related to 0007344 |
| 2025-07-29 20:10 | Jim Luth | Note Edited: 0023173 | |
| 2025-07-29 20:11 | Jim Luth | Note Edited: 0023173 | |
| 2025-08-22 13:57 | Jim Luth | Note Added: 0023242 |