DataTypes Comments Section 1
Editorial Comments on DataTypes Abstract Specification V2.0 Rev 5699
- 1.1 Notes to Readers
- In general, these paragraphs are undecipherable to an outsider. We assume that they will be removed in the final copy?
- 1.2 Acknowledgements
- 1.3 Changes made to this Specification
- Again, these will be going away in the final document?
- 1 Overview
- 1.1 Introduction and Scope
- P3 - ...intended to implemented directly based on the details.
- P4 What is a "payload"?
- RMO-ODP reference won't be obvious to everyone. Recommend an external reference and a short statement about the "external" and "computational" views are.
- 1.2 What is a Data Type?
- P1 - What is a data element and how does it relate to types and values"
- P3 - "A semantic property of a data type is referred to by a name and can be evaluated for any value of data type."
- (Editorial) of a data type?
- Should "semantic property" be defined before we talk about how it fits. In general this paragraph doesn't make a lot of sense. Expand
- P4 - ... construct
any>higher order meanings: messages, computerized patient record documents, or business objects and their transactions
- P4 - Neither identity nor state nor chang
inge of state is defined for a data value. Conversely iI business objects, documents and messages (???), we track state...
- 1.1 Introduction and Scope
are can be defined - intensionally - ... (why the hyphens?) Are cardinal numbers defined as a data type?
1.3 How about "For example, a data type for cardinal numbers (non-negative integers) can be defined as the combination of a successor function, which yields the successor for any given cardinal number plus the rule that zero is the cardinal number that is not the successor for any other cardinal value. We can use this definition...
...the boolean data type can be defined extensionally, with two distinct values.
- going seriously passive here - how about "Data types specify the properties of data values."
- Units aren't the most common example.
- More generally, however, one should view ...
- "value set" isn't defined.
- "Whether semantic properties..." - what is a semantic property???
In general, this section needs some work - expanding and polishing, but it is getting closer.
- "... version 3 is its
openness towardsneutrality regarding representation and implementation technologies. . All HL7 version 3 specifications are supposed to be done in a form independent from specific representation and implementation technologiesrequired to be technology and representationally nutral.
- Some of these questions make no sense until you've read the rest of the spec. If you are going to introduce ANY in the introductory material, you need more explanation.
- With a semantic specification - What IS this?
- NULL comment on BL a bit out of place, but it passes.
- COMP and CEQ seem out of place. In addition, is CEQ abstract as well?
- Instead of the data, an ED may contain only a reference - This needs clarification.
- Note that ST is a specialization of ED where the mediaType is fixed to text/plain and several other properties are constrained to null. - out of place - this goes with the next entry as a second sentence. ST is a specialization of ED ...
- that optionally may - isn't this a split inifinitive?
- CD - this needs redefinition - the definition is not correct and is unnecessarily inflammatory
- CO -
Coded dataA Concept Descriptor where an ordering relation is defined among some or all of the concept codes. Numerical value has *nothing* to do with this. This is implementation specific.
- CS - Coded data where all the fields are predetermined with the exception of the code.
- TEL - is this a URI or a URL? Is URL implementation specific?
- EN is underspecified, but will pass
- PQ - how does this align with the previous 3? - Are INT, REAL and RTO al quantities if so, it would be good to stress this, so we understand that PQ takes one of these and adds dimension.
- EXPR - does this belong in Quantity or the paragrpah below.
- Quantity Collections
- expressional is a wierd word - does it mean expression?
- "Term" is capitalized and used without definitions. Can we fit it into QSET?
- QSD and QSP need to be differentiated
- QSC - need more detail.
- IVL - same
- What is a "generic data type extension"
- "used to specify"? - That specifies?
- " (§ ) are not fully qualified types, but only restrictions on previously defined types and flavors." - what is this? What does it reference...
Something is missing here - shouldn't there be an introductory paragraph of some sort? The paragraph needs to mention that the meaning of the constraints will be clarified by looking at each individual type.
- ED.DOC.INLINE so the contents are a document
is not provided by means of a reference.
- ED.DOC.REF - so that the contents are a reference to a document."
- TEL.URL - didn't the definition of "TEL" say it had to be a URL?
- PN, ... - why is Entity and Person capitalized? Maybe explain in the introductory material? I would rather see lower case.
- IVL.LOW, IVL.HIGH, IVL.WIDTH, ... - not a semantic explanation. Replace lowClosed with what it means.
- GTS.BOUNDEDPIVL - good lord. This needs to be restated in English.
- Should there be an introductory paragraph that grounds the meaning of "SHALL", "MAY" and any other critical keyword?
- All instances of these data types SHALL be valid with respect to the invariant statements contained in this specification.
- Question: Is it up to the creator or the recipient to make sure this is true? The second sentence weakly implies the creator, but it isn't absolutely clear.
- "Definition 1" - not sure I agree with the caption. Maybe this should be a simple textbox?
1.8.1 Constraining Model
- Need a bit more datail.
- Why is "Contstraining Model" capitalized, but not "Primary?
- This paragraph needs clarification - are null flavors outside the scope of constraining models? Is there any way that a constraining model can declare that it doesn't allow a null flavor in a given data value? This tends to imply not, which I don't think is what is intended.
- NI - need to spell this out.
- Where did "*" come in on cardinalities - did we intend to add "0..*" and 1..*"? If not, what goes on. What does "Cardinality of 1" mean with respect to what was just stated. Wouldn't this be more simply put in terms of min and max cardinality?
- Paragraphs appear out of order.
- In general, THIS SECTION NEEDS WORK