Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Jan 19, 2024

Backport of #6964 to release/3.0

/cc @ericstj

Test only infrastructure change.

@codecov
Copy link

codecov bot commented Jan 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9e500a5) 68.80% compared to head (8bd2265) 68.80%.

Additional details and impacted files
@@               Coverage Diff               @@
##           release/3.0    #6967      +/-   ##
===============================================
- Coverage        68.80%   68.80%   -0.01%     
===============================================
  Files             1240     1240              
  Lines           249652   249652              
  Branches         25511    25511              
===============================================
- Hits            171776   171773       -3     
- Misses           71276    71278       +2     
- Partials          6600     6601       +1     
Flag Coverage Δ
Debug 68.80% <ø> (-0.01%) ⬇️
production 63.27% <ø> (-0.01%) ⬇️
test 88.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 3 files with indirect coverage changes

@ericstj
Copy link
Member

ericstj commented Jan 19, 2024

New failure mode showing up in Ubuntu test build:

[Long Running Test] 'Microsoft.ML.Tests.Scenarios.Api.CookbookSamples.CookbookSamplesDynamicApi.TextFeaturization', Elapsed: 00:58:11
[Long Running Test] 'Microsoft.ML.Tests.Transformers.WordEmbeddingsTests.TestWordEmbeddings', Elapsed: 00:58:08

Could there be a deadlock between these two? On my machine they finish in under a minute.

@ericstj ericstj merged commit 9fcaafc into release/3.0 Jan 22, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Feb 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants