Skip to content

Conversation

frankfliu
Copy link
Contributor

Description

Brief description of what this PR is about

  • If this change is a backward incompatible change, why must this change be made?
  • Interesting edge cases to note here

@frankfliu frankfliu requested review from a team and zachgk as code owners September 4, 2024 22:20
@codecov-commenter
Copy link

codecov-commenter commented Sep 4, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 72.60%. Comparing base (5c418be) to head (d48d2bf).
Report is 36 commits behind head on master.

Files with missing lines Patch % Lines
.../src/main/java/ai/djl/ml/xgboost/XgbNDManager.java 60.00% 1 Missing and 1 partial ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3453      +/-   ##
============================================
+ Coverage     72.57%   72.60%   +0.02%     
- Complexity     7377     7389      +12     
============================================
  Files           669      671       +2     
  Lines         32658    32787     +129     
  Branches       3474     3483       +9     
============================================
+ Hits          23701    23804     +103     
- Misses         7319     7341      +22     
- Partials       1638     1642       +4     

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

@frankfliu frankfliu merged commit 3e936cf into deepjavalibrary:master Sep 5, 2024
5 checks passed
@frankfliu frankfliu deleted the xgb branch September 5, 2024 23:31
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.

3 participants