Skip to content

Packaging cleanup #6939

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.

Sign up for GitHub

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 18 commits into from
Feb 28, 2024
Merged

Packaging cleanup #6939

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
db15f47
Packaging cleanup
ericstj Jan 5, 2024
ca66f2b
Apply code review feedback
ericstj Jan 8, 2024
9a9b26b
Suppress copying indirect references
ericstj Jan 9, 2024
7828ea7
Remove unwanted bundled files from AutoML
ericstj Jan 10, 2024
86ff6b5
Remove leading slash
ericstj Jan 10, 2024
1ca51b3
Refactor model download
ericstj Jan 10, 2024
fce6534
Correct the packaging path of native symbols
ericstj Jan 10, 2024
0f22e08
Merge branch 'main' of https://github.com/dotnet/machinelearning into…
ericstj Jan 23, 2024
164d311
Rename NoTargets projects from csproj to proj
ericstj Jan 26, 2024
0489b9f
Merge branch 'main' of https://github.com/dotnet/machinelearning into…
ericstj Jan 26, 2024
fe9a773
Fix build issues around model download and respond to feedback
ericstj Jan 29, 2024
8d795f7
Remove NoTargets file extension enforcement
ericstj Jan 30, 2024
fc8895d
Merge branch 'main' into cleanupSnupkg
ericstj Feb 7, 2024
94b96b4
Merge branch 'main' into cleanupSnupkg
ericstj Feb 7, 2024
eea5f83
Rename proj to CSProj, include in SLN
ericstj Feb 13, 2024
d957285
Merge branch 'main' into cleanupSnupkg
ericstj Feb 14, 2024
498fc84
Merge remote-tracking branch 'upstream/main' into cleanupSnupkg
ericstj Feb 22, 2024
59f414d
Respond to PR feedback
ericstj Feb 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions Directory.Build.targets
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -108,4 +108,11 @@
BeforeTargets="Test"
DependsOnTargets="GenerateCoverageResult"
Condition="'$(IsUnitTestProject)' == 'true' AND '$(TargetFramework)' != '' AND '$(CollectCoverage)' == 'true'" />

<Target Name="ValidateNoTargetsProjectExtension"
BeforeTargets="PrepareForBuild"
Condition="'$(UsingMicrosoftNoTargetsSdk)' == 'true'">
<!-- Make sure any NoTargets projects use the proj extension so that they don't get language specific behavior from other SDKs -->
<Error Text="The project file should have a .proj extension since it uses the NoTargets SDK." Condition="'$(MSBuildProjectExtension)' != '.proj'" />
</Target>
</Project>
4 changes: 2 additions & 2 deletions docs/samples/Microsoft.ML.Samples.GPU/Microsoft.ML.Samples.GPU.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,14 +54,14 @@
</ItemGroup>

<ItemGroup>
<Content Include="$(ArtifactsObjDir)DnnImageModels\ResNet18Onnx\ResNet18.onnx">
<Content Include="$(ModelDownloadPath)\ResNet18Onnx\ResNet18.onnx">
<Link>DnnImageModels\ResNet18Onnx\ResNet18.onnx</Link>
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
</Content>
</ItemGroup>

<ItemGroup>
<Content Include="$(ArtifactsObjDir)DnnImageModels\ResNetPrepOnnx\ResNetPreprocess.onnx">
<Content Include="$(ModelDownloadPath)\ResNetPrepOnnx\ResNetPreprocess.onnx">
<Link>DnnImageModels\ResNetPrepOnnx\ResNetPreprocess.onnx</Link>
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
</Content>
Expand Down
4 changes: 2 additions & 2 deletions docs/samples/Microsoft.ML.Samples/Microsoft.ML.Samples.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -981,14 +981,14 @@
</ItemGroup>

<ItemGroup>
<Content Include="$(ArtifactsObjDir)DnnImageModels\ResNet18Onnx\ResNet18.onnx">
<Content Include="$(ModelDownloadPath)\ResNet18Onnx\ResNet18.onnx">
<Link>DnnImageModels\ResNet18Onnx\ResNet18.onnx</Link>
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
</Content>
</ItemGroup>

<ItemGroup>
<Content Include="$(ArtifactsObjDir)DnnImageModels\ResNetPrepOnnx\ResNetPreprocess.onnx">
<Content Include="$(ModelDownloadPath)\ResNetPrepOnnx\ResNetPreprocess.onnx">
<Link>DnnImageModels\ResNetPrepOnnx\ResNetPreprocess.onnx</Link>
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
</Content>
Expand Down
2 changes: 1 addition & 1 deletion src/Directory.Build.props
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
<Import Project="..\Directory.Build.props" />

<PropertyGroup>
<GenerateDocumentationFile>true</GenerateDocumentationFile>
<GenerateDocumentationFile Condition="'$(UsingMicrosoftNoTargetsSdk)' != 'true'">true</GenerateDocumentationFile>
<!-- Not all types and members have XML docs yet.
When you are ready to tackle adding them, remove <NoWarn>
below, and when you get clean, also remove <WarningsNotAsErrors>
Expand Down
4 changes: 1 addition & 3 deletions src/Microsoft.ML/Microsoft.ML.proj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,11 @@
<!-- We need ProjectReferences to provide output since we're bundling the output -->
<NoTargetsDoNotReferenceOutputAssemblies>false</NoTargetsDoNotReferenceOutputAssemblies>
<!-- Only include directly referenced projects -->
<_FindDependencies >false</_FindDependencies >
<_FindDependencies>false</_FindDependencies>
<!-- Framework references are required for APICompat -->
<DisableImplicitFrameworkReferences>false</DisableImplicitFrameworkReferences>
<!-- This must be set to include any PDBs -->
<IncludeBuildOutput>true</IncludeBuildOutput>
<!-- Don't produce nor include xml docs -->
<GenerateDocumentationFile>false</GenerateDocumentationFile>
<NoWarn>$(NoWarn);NU5127;NU5128</NoWarn>
<IsPackable>true</IsPackable>
<PackageDescription>ML.NET is a cross-platform open-source machine learning framework which makes machine learning accessible to .NET developers.</PackageDescription>
Expand Down
4 changes: 2 additions & 2 deletions test/Microsoft.ML.AutoML.Tests/Microsoft.ML.AutoML.Tests.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,11 +44,11 @@


<ItemGroup>
<Content Include="$(ArtifactsObjDir)DnnImageModels\ResNetPrepOnnx\ResNetPreprocess.onnx">
<Content Include="$(ModelDownloadPath)\ResNetPrepOnnx\ResNetPreprocess.onnx">
<Link>DnnImageModels\ResNetPrepOnnx\ResNetPreprocess.onnx</Link>
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
</Content>
<Content Include="$(ArtifactsObjDir)DnnImageModels\ResNet18Onnx\ResNet18.onnx">
<Content Include="$(ModelDownloadPath)\ResNet18Onnx\ResNet18.onnx">
<Link>DnnImageModels\ResNet18Onnx\ResNet18.onnx</Link>
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
</Content>
Expand Down
4 changes: 2 additions & 2 deletions test/Microsoft.ML.OnnxTransformerTest/Microsoft.ML.OnnxTransformerTest.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,14 @@
</ItemGroup>

<ItemGroup>
<Content Include="$(ArtifactsObjDir)DnnImageModels\ResNetPrepOnnx\ResNetPreprocess.onnx">
<Content Include="$(ModelDownloadPath)\ResNetPrepOnnx\ResNetPreprocess.onnx">
<Link>DnnImageModels\ResNetPrepOnnx\ResNetPreprocess.onnx</Link>
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
</Content>
</ItemGroup>

<ItemGroup>
<Content Include="$(ArtifactsObjDir)DnnImageModels\ResNet18Onnx\ResNet18.onnx">
<Content Include="$(ModelDownloadPath)\ResNet18Onnx\ResNet18.onnx">
<Link>DnnImageModels\ResNet18Onnx\ResNet18.onnx</Link>
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
</Content>
Expand Down