-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Format of Orchestra XML file has changed substantially between EP269 and EP272 #30
Comments
@outofphase, some changes were intentional (see GTC Blog) but the reformatting of the DC terms section was not. I had also detected the issue with Tablature and added issue #67. Don is looking into that. Orchestra XML is now generated from Unified with Python instead of XSLT. |
I fix has already been committed for the extra newline in the metadata section. (Trims leading and trailing whitespace from Unified.) |
Thank you Hanno and Don. I will try to dig a bit deeper into the issues that I found with Tablature then. These are all with elaboration sections that span multiple lines. I think it has to do with duplication of data between documentation sections with purpose SYNOPSIS and with purpose ELABORATION. The end result is that I can't now, with the new XML format, get a stable roundtrip: md -> xml -> md. A new set of documentation lines gets generated in the process. The new XML format looks better, but it is creating some strange results for me which I need to dig into. In EP269 we had:
followed by:
Now in EP272 we have:
followed by:
|
@outofphase you are quoting two different sections of the Orchestra XML file, there is no duplication:
The second snippet you quote above is missing the beginning for which it is an annotation (always comes at the end of a message definition): <fixr:message name="ExecutionReport" id="9" category="SingleGeneralOrderHandling" added="FIX.2.7" abbrName="ExecRpt" msgType="8"> The extra newline issue remains and is being looked into by @donmendelson. |
In the Unified Repository phrases file, a To control format, it would useful to turn those numbered lists into markdown in Orchestra when we are no longer dependent on conversion from Unified. |
Tablature behavior needs to be tested:
|
I also found that the initial |
@JThoennes you are correct that the appinfo/fixml schema has not yet been made public. It is used internally to mark elements for FIXML schema generation. The appinfo element is extensible with
The lax processing is supposed to inform an XML processor that if the schema cannot be found, no errors should be reported. Your experience may be different. The lack of xml declaration is an error; we will look into it. |
@JThoennes the XML namespace reference is not a URL, but a URI to establish a unique identifier for the prefix. We simply chose the resource name to look like a URL. |
@kleihan My IntelliJ IDE marks this URI as an error because it cannot be downloaded. I will try to register it locally. I currently assume that the IDE has a good understanding of XML and report any findings here. |
The format of the FIX Latest Orchestra XML file has changed substantially between EP269 and EP272, see below for snippet. This has caused various issues in my build process, and there is possibly an incompatibility of this new format with the tablature tools. Before I dig into this much further, can I ask whether this change was intentional?
The text was updated successfully, but these errors were encountered: