Skip to content

ignore labels in inference CSV data#652

Merged
farhadrgh merged 4 commits into
mainfrom
farhadr/ignore_labels
Jan 23, 2025
Merged

ignore labels in inference CSV data#652
farhadrgh merged 4 commits into
mainfrom
farhadr/ignore_labels

Conversation

@farhadrgh
Copy link
Copy Markdown
Collaborator

Description

Type of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactor
  • Documentation update
  • Other (please describe):

CI Pipeline Configuration

Configure CI behavior by applying the relevant labels:

Note

By default, the notebooks validation tests are skipped unless explicitly enabled.

Usage

TODO: Add code snippet

Pre-submit Checklist

  • I have tested these changes locally
  • I have updated the documentation accordingly
  • I have added/updated tests as needed
  • All existing tests pass successfully

Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
@farhadrgh farhadrgh marked this pull request as ready for review January 23, 2025 17:59
Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
@farhadrgh farhadrgh enabled auto-merge January 23, 2025 18:52
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.79%. Comparing base (91d3f16) to head (5edf4c4).

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #652   +/-   ##
=======================================
  Coverage   86.79%   86.79%           
=======================================
  Files         118      118           
  Lines        7056     7058    +2     
=======================================
+ Hits         6124     6126    +2     
  Misses        932      932           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

farhadrgh and others added 2 commits January 23, 2025 11:31
Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
Copy link
Copy Markdown
Contributor

@holgerroth holgerroth 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.

@farhadrgh farhadrgh added this pull request to the merge queue Jan 23, 2025
Merged via the queue into main with commit 00391df Jan 23, 2025
@farhadrgh farhadrgh deleted the farhadr/ignore_labels branch January 23, 2025 22:17
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.

5 participants