refactor: Modernize samples by Kryptos-FR · Pull Request #2664 · stride3d/stride

Kryptos-FR · 2025-03-07T13:46:03Z

PR Details

Modernize the samples.

I did not enable the nullable feature since it isn't the case yet in the whole codebase. I also excluded the C# tutorials projects (beside using file scope namespaces) because we might to use a different style for them that is more beginner friendly.

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.

…s the visibility appropriately

* .NET 6 reference missed * added central version number from Jkao for better consistancy * updated comment * Get version from Environment

…an machine (2. attempt, issue stride3d#2232) (stride3d#2319)

…pport, and fix model skinning on platforms using OpenGL ES (ie. Android and iOS) (stride3d#2323) Co-authored-by: Basewq <Basewq@users.noreply.github.com>

* prevents a null IMemberNode to update * checking properties before calling update

* feat: Adjust freeimage to all platforms * Adjust FreeImage methods

…fault animation preview model (stride3d#2291) * ported https://github.com/assimp/assimp/pull/5494/files * Cleanup * Potential fix to investigate further * Fallback to BaseColor when diffuse is not available * Automatically set preview model on import for animations * Use BaseColor as Diffuse if Diffuse is not defined * Document pivot todo * Rollback transformations related logic * More cleanup * Fix issue when trying to import animation from an fbx without animations --------- Co-authored-by: Eideren <contact@eideren.com>

…or lightweight, abstract and typesafe processing in idiomatic c# (stride3d#2331) * feat: FlexibleProcessing as alternative to EntityComponentProcessor for lightweight, abstract and typesafe processing in idiomatic c# * is ... == false to is not * Access backing buffer of the list to improve get/set operations * More tests for edge cases

…ride3d#2337) This reverts commit f23ce73.

Co-authored-by: Basewq <Basewq@users.noreply.github.com>

…#2638) Co-authored-by: Basewq <Basewq@users.noreply.github.com>

…s too small to fit even one character. (stride3d#2598) Signed-off-by: net2cn <mcopener@gmail.com>

… out of sync issues (stride3d#2629)

…ssemblyProcessor (stride3d#2639)

* [Core.Mathematics] Modernize code * [Core.Mathematics] Add tests to Stride.Tests.Simple * [Core] Cleanup some code * [Core.CompilerServices] Cleanup some code * [Core.CompilerServices] Add tests to Stride.Tests.Simple * [Core.IO] Modernize code * [Core.MicroThreading] Modernize code * [Core.Serialization] Modernize code * [Core.AssemblyProcessor.Tests] Modernize code * [Core.CompilerServices.Tests] Modernize code * [Core.Mathematics.Tests] Modernize code * [Core.Tests] Modernize code * [Core.Translation] Modernize code * [Core.Reflection] Modernize code * Fixes build of Stride.VisualStudio.sln * [Core.Yaml.Tests] Modernize code * [Core.Design] Modernize code * [Core.Design.Tests] Modernize code * [Core.Mathematics] More annotations * Fix build of projects that import code from Core.Design Especially those targeting .NET Framework

…ue with Avalonia. (stride3d#2651)

…#2650) * handle orientation, Add SetPose, and fix center of mass issue with non default rotation

…tride3d#2654) * [Core.Presentation] Modernize code * [Core.Design] Make ToEnumerable extension safer It will not throw a cast exception for a type mismatch, but instead filter out bad items. * [Core.Quantum] Modernize code * [Core.Quantum.Tests] Modernize code * [Core.Presentation.Quantum] Modernize code * [Core.Presentation.Quantum.Tests] Modernize code

* [Tests] Allow running tests on Linux * [Core] Run tests on Linux * [Core.Presentation] Run tests on Linux * [Core.Assets] Run tests on Linux * [BuilEngine] Run tests on Linux

…3d#2658) * [Core.BuildEngine.Common] Modernize code * [Core.Packages] Modernize code * [Core.Assets.Yaml] Modernize code * [Build] Remove obsolete Prefer32Bit * [Core.Assets] Modernize code * [Core.Assets.Tests] Modernize code * [Core.Assets.Quantum] Modernize code * [Core.Assets.Quantum.Tests] Modernize code * [xunit.runner.stride] Modernize code * Code review

Kryptos-FR · 2025-03-07T17:35:25Z

We have a problem: the integrated code editor in the GameStudio is old and doesn't understand modern C#. It's thinking all modern lines of code are incorrect.

For example :

Therefore, we can't modernize the samples before we update (or get rid of) the code editor.

IXLLEGACYIXL · 2025-03-07T17:42:18Z

the code editor uses roslyn of net6 the latest roslynpad would understand it

but updating it isnt as easy..

xen2 · 2026-07-17T01:30:17Z

👋 Apologies for the noise here.

This PR was closed unintentionally by a bad force-push to master on my side, which auto-closed a batch of PRs. GitHub won't let me (or you) reopen it, since the close is pinned to the now-rewritten commit.

I've contacted GitHub Support to try reopening these in bulk, but no guarantees. If this change is still relevant, it's a good opportunity to sync your branch with the latest master and open a fresh PR (your code is intact).

Sorry again for the disruption! 🙏

Eideren and others added 30 commits June 10, 2024 22:09

Handle newly introduced ModelGizmo post merge

2a60010

Merge branch 'refs/heads/gizmo' into bepu_integration

9b0fc5d

Specify that changes to colliders may reset internal physics state

fcd662a

Remove CharacterComponent's debug info

3849300

Minor changes and more annotate one more callsite to check with Norbo

b05eeb3

Rewrite constraints backend, introduce Constraint list for BodyComponent

20a8aa2

Create Test project covering BepuPhysics, starting with constraints

c776c43

Fix ListOfColliders workaround, in turn fixing some issues with gizmos

358afca

Basic Constraint Gizmos

4cfa3b6

Enable/Disable gizmos

6af3b24

Remove deprecated Stride.Physics reference

057dbf7

Rename Stride.BepuPhysics.DebugRender to Debug

6af167c

Update .sdscene based on naming and structure changes

e19a7a1

Rollback to EntityProcessor

9282ed5

Improve collider change tracking

b673069

Ensure adding the debug component before adding collidables still set…

f97344a

…s the visibility appropriately

Make FastList Obsolete (stride3d#2332)

ab954ea

Fix a couple of .NET 6 reference we missed (stride3d#2330)

a19caf6

* .NET 6 reference missed * added central version number from Jkao for better consistancy * updated comment * Get version from Environment

Fixes assembly load failure when building Stride based project on cle…

45dcf49

…an machine (2. attempt, issue stride3d#2232) (stride3d#2319)

[Graphics] Remove signed blend indices due to dropping OpenGL ES 2 su…

e678fd2

…pport, and fix model skinning on platforms using OpenGL ES (ie. Android and iOS) (stride3d#2323) Co-authored-by: Basewq <Basewq@users.noreply.github.com>

[GameStudio] Prevents a null IMemberNode to update (stride3d#2307)

5558558

* prevents a null IMemberNode to update * checking properties before calling update

feat: Adjust freeimage to all platforms (stride3d#2303)

15d7dc3

* feat: Adjust freeimage to all platforms * Adjust FreeImage methods

[Editor] Disable ModelComponent Gizmos by default (stride3d#2321)

f1bbc4e

feat: Adjust DirectTex for all platforms (stride3d#2314)

f23ce73

Revert "feat: Adjust DirectTex for all platforms (stride3d#2314)" (st…

2556707

…ride3d#2337) This reverts commit f23ce73.

feat: Adjust DirectTex to all platforms 2 (stride3d#2338)

10babd2

[Editor] Fix displaying & editing project User Settings (stride3d#2350)

dbb125e

Co-authored-by: Basewq <Basewq@users.noreply.github.com>

fix: Update Stride.Core.Assets.CompilerApp.targets file (stride3d#2360)

9c15228

Basewq and others added 19 commits February 15, 2025 22:43

[Android] Fix registering broadcast receiver on Android 14+ (stride3d…

e177ec5

…#2638) Co-authored-by: Basewq <Basewq@users.noreply.github.com>

fix: TextBlock enters an infinite loop if the given available space i…

48701e2

…s too small to fit even one character. (stride3d#2598) Signed-off-by: net2cn <mcopener@gmail.com>

fix: Add assets cleanup target when calling dotnet clean, clarify ref…

2087a58

… out of sync issues (stride3d#2629)

refactor: Nullable and Modernization for Stride.Core. & Stride.Core.A…

44f4ff6

…ssemblyProcessor (stride3d#2639)

fix: Rendering hang when creating object node (stride3d#2640)

2080823

feat: Add asset support for interface properties (stride3d#2606)

324cf36

fix: Temporarily disable the resolver UI as there is a dependency iss…

e68e4d7

…ue with Avalonia. (stride3d#2651)

fix: Ensure teleporting bodies notify bepu's data structure (stride3d…

0d0c656

…#2650) * handle orientation, Add SetPose, and fix center of mass issue with non default rotation

fix: Restore Nuget Resolver UI (stride3d#2653)

8fd8a90

feat: Run core tests on Linux (stride3d#2659)

b12c52a

* [Tests] Allow running tests on Linux * [Core] Run tests on Linux * [Core.Presentation] Run tests on Linux * [Core.Assets] Run tests on Linux * [BuilEngine] Run tests on Linux

feat: Enable to build WPF projects outside windows (stride3d#2661)

dd49a56

Enable implicit usings and latest language version

c679e5b

Removed unused usings and use file scoped namespaces

770361e

Use target-typed new and collection expressions

d670f15

Various code quality improvements

17fe9ee

Add missing icon for FirstPersonShooter

34f085c

Kryptos-FR changed the title ~~Feature/modernize samples~~ refactor: Modernize samples Mar 7, 2025

Kryptos-FR added the area-Samples label May 26, 2025

xen2 force-pushed the master branch 2 times, most recently from ab329f3 to 482bd28 Compare April 16, 2026 07:56

xen2 closed this Jul 16, 2026

xen2 force-pushed the master branch from b9fbd2e to 3d208f6 Compare July 16, 2026 22:11

xen2 added this to Force-push PR recovery (2026-07-16) Jul 23, 2026

xen2 moved this to WIP - re-PR if resuming in Force-push PR recovery (2026-07-16) Jul 23, 2026

Navigation Menu

Uh oh!

refactor: Modernize samples - #2664

refactor: Modernize samples#2664
Kryptos-FR wants to merge 3604 commits into
stride3d:masterfrom
Color-Rise:feature/modernize-samples

Kryptos-FR commented Mar 7, 2025 •

edited

Loading

Uh oh!

Kryptos-FR commented Mar 7, 2025

Uh oh!

IXLLEGACYIXL commented Mar 7, 2025 •

edited

Loading

Uh oh!

xen2 commented Jul 17, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Uh oh!

Conversation

Kryptos-FR commented Mar 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Details

Types of changes

Checklist

Uh oh!

Kryptos-FR commented Mar 7, 2025

Uh oh!

IXLLEGACYIXL commented Mar 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

xen2 commented Jul 17, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Kryptos-FR commented Mar 7, 2025 •

edited

Loading

IXLLEGACYIXL commented Mar 7, 2025 •

edited

Loading