Most names and fields in XUSP have added limits to the string length. There is no limit on the label size. I recommend that it match the mnemonic length of 64 characters.

AlarmMultiRanges feature was added in XTCE 1.2. This feature should not be included in the XUSP 1.1 update since is too complex for the target alarm model.

rangeForm attribute was added in XTCE 1.2. This attribute allows severity level between alarm ranges to be reversed (that is, rangeForm="inside" causes the more severe alarm ranges to inside of less severe alarm ranges). The rangeForm="inside" option not be included in the XUSP 1.1 update since is too complex for the target alarm model.

The XUSProfileRules.csv file contains extraneous spaces between commas. The spacing between commas is inconsistent it's mostly “, ,” but frequently “,,”. These spaces cause non-empty entries when the CSV file is imported into Excel.

Several of the rules appear to have syntax errors, such as:

count(//xtce:CommandMetaData/*/xtce:DefaultAlarm

ParameterToSet/Derivation is a MathOperationType which is abstract. Which means you can make an instance of it.

A simple fix is to make a DerivationType extending MathOperationType.

The following 4 schema-types have funny character encodings for quotes or similar in a few lines of annotation, probably from Word editing.

BinaryArgumentType
BinaryDataType
BinaryParametertype
ServiceRefType

The errors are around "" items like "dangling reference" and some of the path related items.

approx. lines 20, 22, 26 and 58 per each file.

This makes some things complain – maybe compilers, Javadoc other tools.

The quick fix is to simply edit the schema and fix them, then motor on your way.

The following lines have xpath syntax errors and don't "compile" using Java XPath.

Bad #1 rule – @line 830 --> count(//@/@lastEntryForThisArrayInstance)=0
Bad #2 rule – @line 897 --> count(//xtce:CommandMetaData/*/xtce:DefaultAlarm
Bad #3 rule – @line 898 --> count(//xtce:CommandMetaData/*/xtce:ContextAlarmList
Bad #4 rule – @line 939 --> count(//xtce:CommandMetaData/*/xtce:DefaultAlarm
Bad #5 rule – @line 1019 --> count(//xtce:CommandMetaData/*/xtce:DefaultAlarm
Bad #6 rule – @line 1020 --> count(//xtce:CommandMetaData/*/xtce:ContextAlarmList
Bad #7 rule – @line 1044 --> count(//xtce:CommandMetaData/*/xtce:DefaultAlarm
Bad #8 rule – @line 1150 --> count(//xtce:CommandMetaData/*/xtce:DefaultAlarm
Bad #9 rule – @line 1151 --> count(//xtce:CommandMetaData/*/xtce:ContextAlarmList
Bad #10 rule – @line 1201 --> count(//xtce:CommandMetaData/*/xtce:DefaultAlarm
Bad #11 rule – @line 1202 --> count(//xtce:CommandMetaData/*/xtce:ContextAlarmList
Bad #12 rule – @line 1764 --> count(//@/@lastEntryForThisArrayInstance)=0
Bad #13 rule – @line 1805 --> count(//@/@lastEntryForThisArrayInstance)=0
Bad #14 rule – @line 2003 --> count(//@/@lastEntryForThisArrayInstance)=0

Since the VCID is telemetered and is a standard CCSDS construct, suggest that supporting VCID be promoted to being supported first-class instead of via the AncillaryData construct.

Also, SCID and other transfer frame constructs should be similarly supported.

The published specification has extra whitespace in some of the columns. These should be removed before proceeding on updating the rules further.

The XTCE 1.2 specification was approved by the DTC. The XUSP profile, which is based on XTCE, should be updated to use the Elements and Attributes of the XTCE 1.2 schema.