Skip to content

Download and process DrivAerML data for DoMINO using PhysicsNeMo-Curator#1028

Merged
saikrishnanc-nv merged 3 commits intoNVIDIA:mainfrom
saikrishnanc-nv:saikrishnanc/integrate-curator
Aug 4, 2025
Merged

Download and process DrivAerML data for DoMINO using PhysicsNeMo-Curator#1028
saikrishnanc-nv merged 3 commits intoNVIDIA:mainfrom
saikrishnanc-nv:saikrishnanc/integrate-curator

Conversation

@saikrishnanc-nv
Copy link
Contributor

PhysicsNeMo Pull Request

Description

This PR adds instructions to download and process DrivAerML data for DoMINO using [PhysicsNeMo-Curator].

  1. Adds references to PhysicsNeMo-Curator
  2. Removes the AWS download script (Curator has a script for AWS download, and another for Hugging Face download)
  3. Removes the process data script

(https://github.com/NVIDIA/physicsnemo-curator)

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

Copy link
Collaborator

@coreyjadams coreyjadams left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Copy link
Collaborator

@Alexey-Kamenev Alexey-Kamenev left a comment

Choose a reason for hiding this comment

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

LGTM!

@saikrishnanc-nv saikrishnanc-nv force-pushed the saikrishnanc/integrate-curator branch from af94c6d to 7042dc5 Compare August 4, 2025 21:13
@coreyjadams
Copy link
Collaborator

/blossom-ci

@saikrishnanc-nv saikrishnanc-nv merged commit 9655507 into NVIDIA:main Aug 4, 2025
1 check passed
@saikrishnanc-nv saikrishnanc-nv deleted the saikrishnanc/integrate-curator branch August 4, 2025 22: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.

4 participants