math.up() / math.forward() / etc. methods should be properties
还没有人认领这个 Issue。
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 45/100
- Issue 类型
- 重构
- 描述清晰度
- 基本清楚
- 活跃度
- 停滞
- 技术栈
- csharp
调研方向
在仓库中搜索 math.up()、math.forward() 以及其他基轴方法，然后检查它们的调用位置和相关测试。当这些轴值以属性的形式保持一致地暴露，并且现有用法和测试仍然通过时，工作就完成了。
由索引模型根据 Issue 内容生成。
描述
Currently the cardinal axes are defined like so:
[MethodImpl(MethodImplOptions.AggressiveInlining)]
public static float3 forward() { return new float3(0.0f, 0.0f, 1.0f); }
Please consider converting these into properties, ex:
[MethodImpl(MethodImplOptions.AggressiveInlining)]
public static float3 forward => new float3(0.0f, 0.0f, 1.0f);
It is expected within the context of C# that such things should be properties rather than methods.
- 主要语言
- C#
- 星标
- 1.4k
- 派生
- 159
- PR 合并指标
- 30 天内没有已合并 PR
贡献指南
这个仓库没有索引到贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
Unity-Technologies/Unity.Mathematics 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 62/100
-
难度 2/5 1-3 小时 新手友好度 55/100
-
难度 4/5 3-5 天 新手友好度 45/100
-
难度 5/5 一周以上 新手友好度 25/100
Unity-Technologies/Unity.Mathematics#245 · 3 条评论 ·
-
难度 3/5 1-2 天 新手友好度 42/100
Unity-Technologies/Unity.Mathematics#243 · 6 个 reaction ·
查看 Unity-Technologies/Unity.Mathematics 的全部 Issue
相似的 Issue
-
bug
难度 1/5 1 小时以内 新手友好度 90/100
-
Type: enhancement
难度 2/5 1-3 小时 新手友好度 65/100
apache/arrow-adbc#4809 ·
-
type/automation type/tech-debt
难度 2/5 1-3 小时 新手友好度 70/100
-
难度 2/5 1-3 小时 新手友好度 65/100
microsoft/vscode-azurefunctions#5197 · 1 条评论 ·
-
难度 1/5 1 小时以内 新手友好度 85/100
microsoft/microsoft-ui-reactor#1274 ·