Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix charactor controller error #1754

Merged
merged 2 commits into from
Sep 14, 2023
Merged

Conversation

luzhuang
Copy link
Contributor

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

@codecov
Copy link

codecov bot commented Sep 14, 2023

Codecov Report

Patch coverage: 51.25% and project coverage change: -0.85% ⚠️

Comparison is base (b33f3ec) 65.11% compared to head (eb3f674) 64.27%.
Report is 84 commits behind head on main.

❗ Current head eb3f674 differs from pull request most recent head 85b12cd. Consider uploading reports for the commit 85b12cd to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1754      +/-   ##
==========================================
- Coverage   65.11%   64.27%   -0.85%     
==========================================
  Files         386      470      +84     
  Lines       20051    23583    +3532     
  Branches     2869     3352     +483     
==========================================
+ Hits        13057    15158    +2101     
- Misses       5898     7230    +1332     
- Partials     1096     1195      +99     
Files Changed Coverage Δ
packages/core/src/RenderPipeline/Basic2DBatcher.ts 46.21% <0.00%> (-42.02%) ⬇️
packages/core/src/RenderPipeline/RenderContext.ts 100.00% <ø> (ø)
packages/core/src/RenderPipeline/SpriteBatcher.ts 25.88% <0.00%> (-57.26%) ⬇️
...kages/core/src/RenderPipeline/SpriteMaskBatcher.ts 13.55% <0.00%> (-0.24%) ⬇️
...ges/core/src/RenderPipeline/enums/PipelineStage.ts 100.00% <ø> (ø)
packages/core/src/Utils.ts 58.66% <0.00%> (+3.66%) ⬆️
packages/core/src/animation/Keyframe.ts 100.00% <ø> (ø)
...ore/src/animation/animationCurve/AnimationCurve.ts 61.42% <0.00%> (ø)
packages/core/src/asset/AssetType.ts 100.00% <ø> (ø)
packages/core/src/asset/Loader.ts 83.33% <ø> (ø)
... and 219 more

... and 27 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@GuoLei1990 GuoLei1990 added bug Something isn't working physics Engine's physical system labels Sep 14, 2023
@GuoLei1990 GuoLei1990 merged commit 2f8dea7 into galacean:main Sep 14, 2023
5 checks passed
GuoLei1990 added a commit to HypnosNova/engine-1 that referenced this pull request Sep 19, 2023
* main: (23 commits)
  Support entity reference (galacean#1417)
  feat: log physics version (galacean#1769)
  Fix Script destroy bug (galacean#1770)
  Fix collider shape bug (galacean#1768)
  fix: physics unit test timeout (galacean#1766)
  Fix `viewportPointToRay` triggers precision issues when the depth value is 1 (galacean#1767)
  fix: project loader not exported (galacean#1758)
  "v1.1.0-beta.4"
  Opt SpriteBatch code (galacean#1760)
  Fix light shader data abnormal when multiple scenes are activated (galacean#1759)
  Fix BlendShape anim compatible with multiple SkinMeshRenderer in on… (galacean#1756)
  refactor: clear code and opt memory (galacean#1757)
  test: add physx unit timeout (galacean#1755)
  "v1.1.0-beta.3"
  Fix charactor controller error (galacean#1754)
  test: fix unit test timeout
  "v1.1.0-beta.2"
  Fix clone error (galacean#1753)
  Support main thread ktx2 (galacean#1745)
  "v1.1.0-beta.1"
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working physics Engine's physical system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants