Whinarn / UnityMeshSimplifier

Mesh simplification for Unity.
MIT License
1.76k stars 277 forks source link

Simplify options #33

Closed amirebrahimi closed 4 years ago

amirebrahimi commented 4 years ago

Describe the changes Consolidated settings/options in MeshSimplifier to use SimplificationOptions

How to test You can set any of the individual properties as normal or can now set the SimplificationOptions as a block.

Screenshots This was done in order to enable easy exposure of the options in AutoLOD: image

github-actions[bot] commented 4 years ago

:tada: This PR is included in version 2.3.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

amirebrahimi commented 4 years ago

Thanks. Once I noticed your serializable struct it made it quick to support per-simplifier options, which has been a feature request (Unity-Technologies/AutoLOD#7) for some time.

Also, great work on the auto-CI for releases.

Whinarn commented 4 years ago

Thanks. Once I noticed your serializable struct it made it quick to support per-simplifier options, which has been a feature request (Unity-Technologies/AutoLOD#7) for some time.

Ah, cool.

Also, great work on the auto-CI for releases.

I have just started using semantic-release, and I'm still getting adjusted to it. I actually failed twice with this release. First because I totally missed that refactor didn't automatically trigger a release (which makes sense), and then it triggered a release for my revert of that temporary release fix 🤦 Anyways, I'm learning 😄