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

[BUG]: texture Count Exceeded the maximum limit of 7 #304

Closed
Temurayl opened this issue Sep 19, 2023 · 6 comments · Fixed by #364
Closed

[BUG]: texture Count Exceeded the maximum limit of 7 #304

Temurayl opened this issue Sep 19, 2023 · 6 comments · Fixed by #364
Labels
bug Something isn't working

Comments

@Temurayl
Copy link

Temurayl commented Sep 19, 2023

Bug描述

简单明晰的描述问题所在
这个是对文本有限制吗
image

Bug复现流程

通过以下步骤产生bug:

  1. 创建 '...'
  2. 点击 '...'
  3. 删除 '...'
  4. 出现错误 '...'

测试引擎版本:

0.6.9

@Temurayl Temurayl added the bug Something isn't working label Sep 19, 2023
@lslzl3000
Copy link
Collaborator

这个请补充一下代码吧,最好有复现步骤

@Temurayl
Copy link
Author

这个请补充一下代码吧,最好有复现步骤

https://codepen.io/Temurayl/pen/ZEVaWbb?editors=0011
核心代码在里面的for循环中
不知道有没有更好的解决办法,在文档中暂未看到相关需求的一个相关介绍

image
image

@OriIIusion
Copy link
Contributor

我把这几段代码拿出了for循环就能渲染出8个了

let bitmapTexture2D = new BitmapTexture2D()
bitmapTexture2D.flipY = true
await bitmapTexture2D.load('https://cdn.orillusion.com/images/webgpu.png')
let s = makeAloneSprite('webgpu', bitmapTexture2D)

image

@OriIIusion
Copy link
Contributor

我把这几段代码拿出了for循环就能渲染出8个了

let bitmapTexture2D = new BitmapTexture2D()
bitmapTexture2D.flipY = true
await bitmapTexture2D.load('https://cdn.orillusion.com/images/webgpu.png')
let s = makeAloneSprite('webgpu', bitmapTexture2D)

image

前三段代码拿出for循环,
makeAloneSpirte()放在for循环里仍然会报错,
感觉就是makeAloneSpirte()放在for循环里出的错,
具体原因等待大佬来解释。

@hellmor
Copy link
Collaborator

hellmor commented Nov 9, 2023

单个UIPanel实例时会自动创建一个UI相关的材质球,这单个材质球对于用到的贴图有数量限制。
在当前Issue里面,贴图数量超过限制,那么你可以将这些单元分组到不同的UIPanel里面去。

@hellmor
Copy link
Collaborator

hellmor commented Nov 9, 2023

单个UIPanel实例时会自动创建一个UI相关的材质球,这单个材质球对于用到的贴图有数量限制。 在当前Issue里面,贴图数量超过限制,那么你可以将这些单元分组到不同的UIPanel里面去。

当前版本可以先调整业务代码去规避这个限制。在未来,说不定这个问题就优化掉了。

hellmor added a commit to X-OldGentleMan/orillusion that referenced this issue Nov 10, 2023
fix: texture Count Exceeded the maximum limit of 7
Codeboy-cn added a commit that referenced this issue Jan 26, 2024
* fix: fix gltf sample

fix gltf sample
fix emissive color

* chore: fix type

* fix: fix chromaKey material

fix image material
fix chromaKey material

* chore: fix VideoMateial

* fix: fix grass sample

fix grass sample

* fix:  fix resize

fix resize

* fix: fix canvas resize change gpu attachments texture

fix canvas resize change gpu attachments texture

* fix: fix resize events

fix resize events

* fix: fix sample grass

fix sample grass

* fix: resize by ResizeObserver

* add post test

* fix: fix post sample resize bug

fix post sample resize bug

* change model url

* feat: Fix csm shadow.

Fix csm shadow.

* chore: add sample

add sample

* fix: Cancel automatic resizing of rendertexture in GI

Cancel automatic resizing of rendertexture in GI

* feat: add graphic ribbon

add graphic ribbon

* fix: Wrong offset for bloom.

Fix wrong offset for bloom.

* perf: reduce texture sample times.

Reduce texture sample times in GI lightingPass.

* feat: fix issues #304

fix: texture Count Exceeded the maximum limit of 7

* chore: move GodRay setting from post file  to engine setting

Move GodRay setting from post file to engine setting.

* chore: get param from setting.

Get param from setting.

* feat: add new graphic face

add new graphic face

* fix index.ts

* chore: Hide light debug line

* chore: Remove sample GraphicTrailing

* update physics car sample

* chore: modify error information

* fix: Sample LoadGLB

* chore: Remove sample GraphicPath

* update public assets

* fix index import

* fix: class name register error

* fix: Auto sort transparent renderers. (#318)

* perf: Reduce computational complexity

Reduce computational complexity

* feat(graphic3d): add compute graphic

add compute graphic

* feat: add compute face atomic auto add faceIndex

add compute face atomic auto add faceIndex

* feat(shader): add attribute auto location

* feat: add compute graphic

add compute graphic

* feat: Implement basic Shape3D graphics rendering capabilities

Cirlce/Rect/RoundRect/Ellipse

* feat: add LineShape3D

Add line drawing capabilities that support lineJoin types of bevel/miter/round...

* feat: new feature of CurveShape3D .

new feature of CurveShape3D .

* feat: shape order

Support dynamic adjustment of shape order.

* fix: pointIndex in shapeData.

Change the global pointIndex to local pointIndex.

* feat: seperate line and fill area.

Seperate line and fill area of shape3d.

* feat: Implement the startAngle and endAngle capabilities for circles and ellipses

Implement the startAngle and endAngle capabilities for circles and ellipses

* feat: Add Path3D API.

Add Path3D API.

* fix: Error of moveTo api.

Error of moveTo api.

* feat: Implement a 3D path renderer

Implement a 3D path renderer

* feat: Add annotation.

Add annotation.

* Squashed commit of the following:

commit 68e6f56
Author: ShuangLiu <liu.shuang@huasheng.io>
Date:   Thu Jan 11 16:22:25 2024 +0800

* chore: update pnpm-lock.yaml

* docs:Add annotation.

Add annotation.

* fix: Set isClosed for Path2D and Path3D to always be false

Set isClosed for Path2D and Path3D to always be false

* fix: Remove grass graphic renderer.

Remove some samples.

* chore: Update import path.

Update import path.

* fix: Auto bind location block.

Auto bind location block.

* feat: Enable Shape3D to support FillRotation

Enable Shape3D to support FillRotation

* upd: Delete some samples.

Delete some samples.

* perf: Simplified path calculation.

Simplify the calculation of the orientation of each line segment.

---------

Co-authored-by: OldGentleMan <blackswanxn@gmail.com>
Co-authored-by: ShuangLiu <liu.shuang@huasheng.io>
Co-authored-by: Codeboy-cn <zhangcanbo@outlook.com>
@lslzl3000 lslzl3000 mentioned this issue Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants