Skip to content

File-scoped namespaces in files under Data (Microsoft.ML.Core) - #6789

Merged
michaelgsharp merged 2 commits into
dotnet:mainfrom
Lehonti:improvement3
Aug 31, 2023
Merged

File-scoped namespaces in files under Data (Microsoft.ML.Core)#6789
michaelgsharp merged 2 commits into
dotnet:mainfrom
Lehonti:improvement3

Conversation

@Lehonti

@Lehonti Lehonti commented Aug 10, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Aug 10, 2023

Copy link
Copy Markdown

Codecov Report

Merging #6789 (9e980a3) into main (39235a7) will increase coverage by 0.00%.
Report is 1 commits behind head on main.
The diff coverage is 80.27%.

@@           Coverage Diff           @@
##             main    #6789   +/-   ##
=======================================
  Coverage   69.03%   69.03%           
=======================================
  Files        1237     1237           
  Lines      252700   252742   +42     
  Branches    26446    26449    +3     
=======================================
+ Hits       174449   174484   +35     
- Misses      71308    71317    +9     
+ Partials     6943     6941    -2     
Flag Coverage Δ
Debug 69.03% <80.27%> (+<0.01%) ⬆️
production 63.60% <80.27%> (-0.01%) ⬇️
test 88.84% <ø> (+<0.01%) ⬆️

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

Files Changed Coverage
src/Microsoft.ML.Core/Data/ServerChannel.cs 0.00%
src/Microsoft.ML.Core/Data/DataKind.cs 70.73%
src/Microsoft.ML.Core/Data/ProgressReporter.cs 72.17%
src/Microsoft.ML.Core/Data/IFileHandle.cs 74.02%
src/Microsoft.ML.Core/Data/ReadOnlyMemoryUtils.cs 77.77%
src/Microsoft.ML.Core/Data/Repository.cs 80.91%
src/Microsoft.ML.Core/Data/AnnotationUtils.cs 82.75%
src/Microsoft.ML.Core/Data/IEstimator.cs 83.33%
src/Microsoft.ML.Core/Data/ModelLoading.cs 83.72%
src/Microsoft.ML.Core/Data/ModelLoadContext.cs 84.21%
... and 16 more
@michaelgsharp

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).
@michaelgsharp
michaelgsharp merged commit aaf226c into dotnet:main Aug 31, 2023
@Lehonti
Lehonti deleted the improvement3 branch August 31, 2023 07:29
@ghost ghost locked as resolved and limited conversation to collaborators Sep 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

2 participants