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.

Sign up for GitHub

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

Jump to bottom

export oms_snapshot using snapshot interface #971

Merged
merged 1 commit into from
Mar 1, 2021
Merged

export oms_snapshot using snapshot interface #971

merged 1 commit into from
Mar 1, 2021

Conversation

arun3688
Copy link
Contributor

@arun3688 arun3688 commented Mar 1, 2021

Related Issues

#945

Purpose

The purpose of this PR is to export <oms_snapshot> using the snapshot interface class

@arun3688 arun3688 requested a review from lochel March 1, 2021 14:25
lochel
lochel approved these changes Mar 1, 2021
View reviewed changes
Copy link
Member

@lochel lochel left a comment

Choose a reason for hiding this comment

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

This needs more work obviously, because the nodes still get created at the root level and the snapshot object isn't really used. But that might have to wait until the next pull request.

@lochel lochel merged commit b049c8c into OpenModelica:master Mar 1, 2021
@arun3688 arun3688 deleted the exportoms_snapshot branch March 2, 2021 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lochel lochel lochel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@arun3688 @lochel