Skip to content

Update sample project assets - #3360

Open
ds5678 wants to merge 7 commits into
stride3d:masterfrom
ds5678:reimport-samples
Open

Update sample project assets#3360
ds5678 wants to merge 7 commits into
stride3d:masterfrom
ds5678:reimport-samples

Conversation

@ds5678

@ds5678 ds5678 commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

PR Details

This pull request updates the sample projects with reimported assets.

Related Issue

I was preparing to do other pull requests related to the sample projects, and the Game Studio prompted me to reimport a bunch of assets. Apparently, their data format has changed a bit since creation.

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My change requires a change to the documentation.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have built and run the editor to try this change out.

@Kryptos-FR

Copy link
Copy Markdown
Member

I believe this was introduced by #3270 where asset references now include a prefix with the path to the package they originate from.

@Kryptos-FR Kryptos-FR left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A few remarks and questions. Did you make sure to run every single sample?

Comment thread samples/Physics/BepuSample/Assets/Shared/Models/nav_test.sdm3d
Comment thread samples/StrideSamples.slnx Outdated
@Kryptos-FR

Copy link
Copy Markdown
Member

@xen2 this PR made me realize that we don't have CI jobs for the samples.

@ds5678

ds5678 commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

Did you make sure to run every single sample?

I just finished testing after my latest set of changes. I ran every single sample (except VR) and verified their functional behavior.

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