Comment:

Suggest re-reviewing whether overloaded attributes should in fact be separate attributes. E.g., Patient.deceased is allowed to be both boolean and dateTime. It seems like there should be a Patient.deceased with a boolean and a Patient.deceasedDateTime with a dateTime. Not sure the cost (more complicated processing logic, difficulty of mapping to a logical model where the attributes will likely be separate) is worth the benefit (use of a shorthand notation to reduce number of attributes).