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

L-MGN: improve inference #798

Closed
wants to merge 5 commits into from

Conversation

Alexey-Kamenev
Copy link
Collaborator

@Alexey-Kamenev Alexey-Kamenev commented Feb 24, 2025

Modulus Pull Request

Description

Improving Lagrangian MGN example inference:

  • Fixed issue with missing graph update during inference.
  • Added support for more than one sequence to run the inference on.
  • Added support multi-sequence error.png.
  • Added material color mapping like in the original code.
  • Updated README.

Other changes:

  • Added multi-material experiment config.
  • Removed redundant code.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The CHANGELOG.md is up to date with these changes.
  • An issue is linked to this pull request.

Dependencies

Sorry, something went wrong.

@Alexey-Kamenev Alexey-Kamenev added the 4 - In Review Currently Under Review label Feb 24, 2025
@Alexey-Kamenev Alexey-Kamenev self-assigned this Feb 24, 2025
@Alexey-Kamenev
Copy link
Collaborator Author

/blossom-ci

@Alexey-Kamenev
Copy link
Collaborator Author

/blossom-ci

1 similar comment
@Alexey-Kamenev
Copy link
Collaborator Author

/blossom-ci

@Alexey-Kamenev
Copy link
Collaborator Author

Merged in GitLab.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4 - In Review Currently Under Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant