Skip to content

Add IterationDataAttribute #4561

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

Merged
merged 1 commit into from
Dec 10, 2019
Merged

Add IterationDataAttribute #4561

merged 1 commit into from
Dec 10, 2019

Conversation

sharwell
Copy link
Member

No description provided.

@sharwell sharwell requested a review from a team as a code owner December 10, 2019 18:57
using System.Reflection;
using Xunit.Sdk;

namespace Microsoft.ML.TestFrameworkCommon.Attributes
Copy link
Member Author

Choose a reason for hiding this comment

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

We should consider consolidating the namespaces Microsoft.ML.TestFramework and Microsoft.ML.TestFrameworkCommon, but I left that for another change.

@codecov
Copy link

codecov bot commented Dec 10, 2019

Codecov Report

Merging #4561 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #4561      +/-   ##
==========================================
+ Coverage   75.12%   75.13%   +<.01%     
==========================================
  Files         908      909       +1     
  Lines      160229   160240      +11     
  Branches    17250    17252       +2     
==========================================
+ Hits       120372   120392      +20     
+ Misses      35041    35025      -16     
- Partials     4816     4823       +7
Flag Coverage Δ
#Debug 75.13% <0%> (ø) ⬆️
#production 70.53% <ø> (+0.01%) ⬆️
#test 90.28% <0%> (-0.04%) ⬇️
Impacted Files Coverage Δ
...ameworkCommon/Attributes/IterationDataAttribute.cs 0% <0%> (ø)
...rosoft.ML.AutoML/ColumnInference/TextFileSample.cs 59.6% <0%> (-2.65%) ⬇️
...L.AutoML/TrainerExtensions/TrainerExtensionUtil.cs 85.15% <0%> (-1.75%) ⬇️
...soft.ML.Data/DataLoadSave/Text/TextLoaderCursor.cs 84.7% <0%> (-0.61%) ⬇️
...soft.ML.TestFramework/DataPipe/TestDataPipeBase.cs 76.08% <0%> (-0.4%) ⬇️
...ML.Transforms/Text/StopWordsRemovingTransformer.cs 86.1% <0%> (-0.16%) ⬇️
....ML.AutoML/PipelineSuggesters/PipelineSuggester.cs 84.03% <0%> (+0.84%) ⬆️
...soft.ML.Transforms/Text/WordEmbeddingsExtractor.cs 87.52% <0%> (+1.13%) ⬆️
...icrosoft.ML.AutoML/Experiment/SuggestedPipeline.cs 92.78% <0%> (+4.12%) ⬆️
... and 2 more

@sharwell sharwell merged commit d306ac8 into dotnet:master Dec 10, 2019
@sharwell sharwell deleted the iteration-data branch December 10, 2019 22:04
@ghost ghost locked as resolved and limited conversation to collaborators Mar 19, 2022
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