Update fmi-ls-dae.xml - #690
Merged
Merged
Conversation
To match PR modelica/fmi-ls-dae#64
chrbertsch
approved these changes
Aug 27, 2026
chrbertsch
left a comment
chrbertsch
left a comment
Collaborator
There was a problem hiding this comment.
Looks good to me.
Do we have tooling to validate against the schema?
elmir-nahodovic
added a commit
to elmir-nahodovic/fmi-ls-dae
that referenced
this pull request
Aug 27, 2026
Schema: * import the generic FMI layered standard manifest schema * Change name to fmiDAEManifest * keep DAE root and child elements unnamespaced (removed targetNamespace) manifest.adoc: * Remove nominal element after discussion with @t-sommer. Instead a nominal attribute for the `<AlgebraicVariable>` element is added. Compare with modelica/Reference-FMUs#690 * Update root name to fmiDAEManifest
elmir-nahodovic
added a commit
to modelica/fmi-ls-dae
that referenced
this pull request
Sep 2, 2026
* Update manifest In summary: Remove formulation, remove index attribute, update docs, update schema * Update mathematical notation to use `a` instead of `z` for algebraic variables (to not conflict with core standard) * Remove <Formulation> element * Remove (plural, enclosing) <AlgebraicVariables> element (instead <AlgebraicVariable> is just listed under the manifest) * Add <Nominal> element, closing issue #61 * removed index attribute from the residual element (until we have a better proposal for higher index formulations) * Remove annotation TODO. Not sure what should be there. * Updating Hello World Example XMLs * Update schema * Update reference-FMUs fmi-ls-manifest.xml * Update fmi-ls-manifest-example.xml * Iteration on standard and schema Schema: * import the generic FMI layered standard manifest schema * Change name to fmiDAEManifest * keep DAE root and child elements unnamespaced (removed targetNamespace) manifest.adoc: * Remove nominal element after discussion with @t-sommer. Instead a nominal attribute for the `<AlgebraicVariable>` element is added. Compare with modelica/Reference-FMUs#690 * Update root name to fmiDAEManifest * Restore fmi3LayeredStandardManifest * Update fmi3LayeredStandardDaeManifest.xsd * Updated to fit the schema * Removed namespace for root element fmiDAEManifest * One more iteration on schema and examples * Restore algebraic container and other small fixes with xsd * Restore AlgebraicVariables container * Remove nominal element * Update xsd * Update example XMLs * Simplify XSD schema * Add XSD dependencies * Refactor manifest schema * Fix trailing whitespace * Correct sequence ModelStructure XMLs * minOccurs and maxOccurs for AlgebraicVariables and ModelStructure * Add required elements to example xml * Untabify schema file to match FMI conventions * Replace duplicated FMI schemas with github references * Update workflow to handle remote schema references * Fix reference FMU example for new schema * Adjust name of `EnableDAEParameter` element in docs * Name of `enableDAEModeVariable` -> `enableDAEModeParameter` --------- Co-authored-by: Torsten Sommer <torsten.sommer@3ds.com> Co-authored-by: Pierre R. Mai <pmai@pmsf.de>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
To match PR modelica/fmi-ls-dae#64