Skip to content

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

Merged
adrastogi merged 3 commits into
mainfrom
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 into
mainfrom
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

adrastogi commented Feb 28, 2026

Copy link
Copy Markdown
Contributor Author

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

2 participants