Add a GPU-based benchmarking workflow#477
Conversation
Signed-off-by: Eric Shi <ershi@nvidia.com>
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThe changes remove the existing ASV benchmark workflow ( Changes
Sequence Diagram(s)sequenceDiagram
participant GitHubActions
participant Author
participant AWS
participant EC2Instance
participant S3
GitHubActions->>Author: On PR to 'newton-physics/newton', check author membership
GitHubActions->>AWS: Assume IAM role and find latest GPU AMI
GitHubActions->>AWS: Launch EC2 GPU instance
AWS->>EC2Instance: Start instance
GitHubActions->>EC2Instance: Send benchmark script via SSM
EC2Instance->>S3: Upload logs and results
GitHubActions->>S3: Retrieve logs
GitHubActions->>AWS: Terminate EC2 instance (cleanup)
Estimated code review effort🎯 4 (Complex) | ⏱️ ~35 minutes Suggested reviewers
📜 Recent review detailsConfiguration used: .coderabbit.yml 📒 Files selected for processing (4)
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
…ics#477) # Description Fixes in Cosmos-Mimic (Augmented Imitation Learning) documentation: 1. Corrected the command specified for the Mimic data generation step to use CPU instead of GPU. 2. Added link to specific Cosmos example to follow for Cosmos generation/augmentation step. ## Type of change - Bug fix (non-breaking change which fixes an issue) - This change requires a documentation update ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] I have updated the changelog and the corresponding version in the extension's `config/extension.toml` file - [x] I have added my name to the `CONTRIBUTORS.md` or my name already exists there
Description
This replaces the CPU-only ASV workflow with a GPU-based workflow using EC2 instances.
Log-downloading logic has been revised.
Instances have been moved to a
g6e.xlarge.Newton Migration Guide
Please ensure the migration guide for warp.sim users is up-to-date with the changes made in this MR.
docs/migration.rstis up-to dateBefore your PR is "Ready for review"
newton/tests/test_examples.py)pre-commit run -aSummary by CodeRabbit