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

透视表列多时,使用触摸板左右快速滑动,会报栈溢出错误 #2627

Closed
1 of 5 tasks
Create-Peace opened this issue Mar 25, 2024 · 13 comments · Fixed by #2643
Closed
1 of 5 tasks

透视表列多时,使用触摸板左右快速滑动,会报栈溢出错误 #2627

Create-Peace opened this issue Mar 25, 2024 · 13 comments · Fixed by #2643
Assignees
Labels
next 2.0-next 版本的问题 released on @next

Comments

@Create-Peace
Copy link

Create-Peace commented Mar 25, 2024

🏷 Version

Package Version
@antv/s2 next
@antv/s2-react
@antv/s2-vue

Sheet Type

  • PivotSheet
  • TableSheet
  • GridAnalysisSheet
  • StrategySheet
  • EditableSheet

🖋 Description

透视表列比较多,使用触摸板左右滑动快时会出现栈溢出错误, 看着像是@antv/g引擎报错了。

Uncaught RangeError: Maximum call stack size exceeded
    at @antv_s2.js?v=75bfa16a:39272:55
    at @antv_s2.js?v=75bfa16a:10425:18
    at Array.forEach (<anonymous>)
    at SyncHook2.call (@antv_s2.js?v=75bfa16a:10424:22)
    at RenderingService2.render (@antv_s2.js?v=75bfa16a:12154:29)
    at Canvas2.render (@antv_s2.js?v=75bfa16a:16213:24)
    at RootInteraction2.draw (@antv_s2.js?v=75bfa16a:43746:34)
    at RootInteraction2.clearState (@antv_s2.js?v=75bfa16a:43750:14)
    at RootInteraction2.changeState (@antv_s2.js?v=75bfa16a:43769:12)
    at PivotSheet2.<anonymous> (@antv_s2.js?v=75bfa16a:41177:21)

⌨️ Code Snapshots

官网demo向右滑动快的时候就能复现这个问题
https://s2.antv.antgroup.com/examples/case/performance-compare/#pivot

🔗 Reproduce Link

image

🤔 Steps to Reproduce

😊 Expected Behavior

😅 Current Behavior

💻 System information

Environment Info
System
Browser
@Create-Peace Create-Peace changed the title 透视表列多时,左右滑动,会报栈溢出错误 透视表列多时,使用触摸板左右快速滑动,会报栈溢出错误 Mar 26, 2024
@wuhaiyang wuhaiyang assigned wuhaiyang and lijinke666 and unassigned wuhaiyang Mar 27, 2024
@lijinke666
Copy link
Member

未复现

@Create-Peace
Copy link
Author

我使用的是Windows 10, 录了个视频。https://pan.baidu.com/s/141qet3k0MmyH2DeHVIp_wA 提取码: 3wir

@lijinke666
Copy link
Member

没有 windows 电脑, 无法复现, 你可以尝试定位下问题, 来个 PR

@Create-Peace
Copy link
Author

我尝试修复了,没有定位到问题,Mac 也能复现,向右滑动最后。

@DavisYao88 DavisYao88 mentioned this issue Apr 1, 2024
10 tasks
@lijinke666 lijinke666 added the next 2.0-next 版本的问题 label Apr 30, 2024
@EliazTray
Copy link

image
同样也是很多透视列,横向快速滚动时,会循环调用,导致栈溢出。

SpreadSheet 中 emit GLOBAL_MOUSE_MOVE
image

@lijinke666 lijinke666 linked a pull request May 27, 2024 that will close this issue
10 tasks
@lijinke666
Copy link
Member

下周发布

@lijinke666
Copy link
Member

🎉 This issue has been resolved in version @antv/s2-v2.0.0-next.23 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lijinke666
Copy link
Member

🎉 This issue has been resolved in version @antv/s2-react-v2.0.0-next.22 🎉

The release is available on:

Your semantic-release bot 📦🚀

@FreeLiC
Copy link

FreeLiC commented Jun 7, 2024

升级到next.23版本 问题仍然存在 官网的例子即可复现

@DavisYao88
Copy link

DavisYao88 commented Jun 11, 2024

升级到next.23版本 问题仍然存在 官网的例子即可复现

确定是23版本吗,拉取源码尝试,目前没复现。内存曲线也正常
image

image

@FreeLiC
Copy link

FreeLiC commented Jun 11, 2024

@DavisYao88

iShot_2024-06-11_15.35.19.mp4

这是官网示例的复现视频

@xiaogang12131
Copy link

下周发布

hello,这个发布了,我下载最新的next,还是会栈溢出

@lijinke666
Copy link
Member

lijinke666 commented Jul 4, 2024

请勿在已关闭的 issue 继续回复, 问题待修复, 见 #2771 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next 2.0-next 版本的问题 released on @next
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants