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

Update test_IQM.py #1826

Merged
merged 5 commits into from
Jul 10, 2024
Merged

Update test_IQM.py #1826

merged 5 commits into from
Jul 10, 2024

Conversation

BalajiJBcs
Copy link
Contributor

Improvements enhance readability, robustness

Description

Improvements enhance readability, robustness
Copy link

copy-pr-bot bot commented Jun 17, 2024

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Copy link

github-actions bot commented Jun 17, 2024

CLA Assistant Lite bot All Contributors have signed the CLA.

@BalajiJBcs
Copy link
Contributor Author

I have read the Contributor License Agreement and I hereby accept the Terms.

Copy link
Collaborator

@schweitzpgi schweitzpgi left a comment

Choose a reason for hiding this comment

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

LGTM

@schweitzpgi schweitzpgi enabled auto-merge (squash) July 8, 2024 15:53
@schweitzpgi
Copy link
Collaborator

schweitzpgi commented Jul 10, 2024

/ok to test

Command Bot: Processing...

@schweitzpgi schweitzpgi merged commit e5cb018 into NVIDIA:main Jul 10, 2024
125 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jul 10, 2024
@bettinaheim bettinaheim added no release notes Don't list this PR in the release notes testing Relates to testing labels Jul 29, 2024
@bettinaheim bettinaheim added this to the release 0.8.0 milestone Jul 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
no release notes Don't list this PR in the release notes testing Relates to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants