Skip to content
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

Add computed attribute #1679

Merged
merged 12 commits into from
Feb 17, 2022
Merged

Conversation

t-sommer
Copy link
Collaborator

No description provided.

@t-sommer t-sommer marked this pull request as ready for review February 17, 2022 08:32
@andreas-junghanns andreas-junghanns added this to the v3.0-beta.4 milestone Feb 17, 2022
<xs:enumeration value="tunable"/>
<xs:enumeration value="changing"/>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the removal of changing originate from my PR?
It that case I belive it was a mistake

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added that back into your initial PR - @t-sommer must rebase his proposal, I guess...

docs/2_4_common_schema.adoc Show resolved Hide resolved
docs/2_4_common_schema.adoc Show resolved Hide resolved
docs/2_4_common_schema.adoc Show resolved Hide resolved
@andreas-junghanns
Copy link
Contributor

@KarlWernersson : would you accept @t-sommer changes on top of yours?
Can we simplify all of this again and move this into one PR?

Copy link
Collaborator

@KarlWernersson KarlWernersson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@t-sommer t-sommer merged commit e88d1e2 into modelica:master Feb 17, 2022
@t-sommer t-sommer deleted the add-computed-attribute branch February 17, 2022 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants