Skip to content

Windows ML: Add folder partitioning for native samples and fix some build errors#597

Merged
adrastogi merged 3 commits intomainfrom
user/adrastogi/samples-output
Mar 21, 2026
Merged

Windows ML: Add folder partitioning for native samples and fix some build errors#597
adrastogi merged 3 commits intomainfrom
user/adrastogi/samples-output

Conversation

@adrastogi
Copy link
Copy Markdown
Contributor

Description

A longstanding problem with the native Windows ML samples is that they share a common output directory, which results in their outputs overwriting each other (especially when building via the build.cmd script). This change adds a fix to separate those outputs so that each sample's binaries land in a different folder. This will better enable scenarios like pipeline automation.

I also noticed that there were some packages missing from the shine-oss feed which I worked with the WASDK team offline to add. And then there were a couple build errors caused by duplicate packaging and a missed macro in the GenAI project, so I tried to resolve those as well.

Target Release

2.0

Checklist

Note that /azp run currently isn't working for this repo.

Comment thread Samples/WindowsML/cpp/Directory.Build.props Outdated
@adrastogi
Copy link
Copy Markdown
Contributor Author

adrastogi commented Feb 28, 2026

Holding off on merging this one for now; there's some deviation between this branch and release/experimental which needs to be reconciled as it relates to building the samples.

[Edit: I think it should be ok to merge these changes after all, so I'll proceed with that.]

@adrastogi adrastogi merged commit aec9442 into main Mar 21, 2026
4 checks passed
@adrastogi adrastogi deleted the user/adrastogi/samples-output branch March 21, 2026 01:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants