Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

michaelgsharp
Copy link
Contributor

Added in support for VS 2022 in the native build.

Also added documentation stating the version of CMake needed for 2022.

@codecov
Copy link

codecov bot commented Oct 5, 2021

Codecov Report

Merging #5956 (a4d6ad7) into main (6ae2cff) will increase coverage by 0.04%.
The diff coverage is 78.63%.

@@            Coverage Diff             @@
##             main    #5956      +/-   ##
==========================================
+ Coverage   68.19%   68.23%   +0.04%     
==========================================
  Files        1142     1142              
  Lines      242367   242803     +436     
  Branches    25355    25385      +30     
==========================================
+ Hits       165287   165686     +399     
- Misses      70400    70421      +21     
- Partials     6680     6696      +16     
Flag Coverage Δ
Debug 68.23% <78.63%> (+0.04%) ⬆️
production 62.89% <78.63%> (+0.01%) ⬆️
test 88.61% <ø> (+0.04%) ⬆️

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

Impacted Files Coverage Δ
...rosoft.Data.Analysis/ArrowStringDataFrameColumn.cs 63.54% <0.00%> (-0.47%) ⬇️
...crosoft.Data.Analysis/DataFrame.BinaryOperators.cs 19.64% <ø> (ø)
src/Microsoft.Data.Analysis/DataFrame.cs 86.68% <ø> (ø)
...t.Data.Analysis/DataFrameColumn.BinaryOperators.cs 5.55% <ø> (ø)
...lysis/PrimitiveDataFrameColumn.BinaryOperations.cs 40.30% <ø> (ø)
...imitiveDataFrameColumn.ReversedBinaryOperations.cs 50.00% <ø> (ø)
...ata.Analysis/PrimitiveDataFrameColumnArithmetic.cs 49.49% <ø> (ø)
...Analysis/StringDataFrameColumn.BinaryOperations.cs 79.41% <ø> (ø)
src/Microsoft.Data.Analysis/Strings.Designer.cs 43.26% <0.00%> (-0.95%) ⬇️
src/Microsoft.Extensions.ML/PoolLoader.cs 65.00% <ø> (ø)
... and 268 more

Copy link
Member

@tarekgh tarekgh left a comment

Choose a reason for hiding this comment

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

Added a little comment. LGTM otherwise.

@michaelgsharp michaelgsharp merged commit 54a3cb9 into dotnet:main Oct 5, 2021
@michaelgsharp michaelgsharp deleted the vs2022 branch October 5, 2021 18:02
Copy link
Member

@safern safern left a comment

Choose a reason for hiding this comment

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

Do we have any reason to keep support for VS2015 and VS2017 around?

@michaelgsharp
Copy link
Contributor Author

This is just for building ML.NET. Do you think there is a need to remove the ability to build it on older versions of visual studio? I'm not sure if people are still using older versions or not. Is there a way to see?

@safern
Copy link
Member

safern commented Oct 8, 2021

There is not, it is not a big deal, we just in dotnet/runtime remove support for the old ones where the TFMs we use are not even supported.

@ghost ghost locked as resolved and limited conversation to collaborators Mar 17, 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.

3 participants