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

Maximum call stack size exceeded #2202

Closed
zyl-fz opened this issue Dec 21, 2023 · 10 comments · Fixed by #2206
Closed

Maximum call stack size exceeded #2202

zyl-fz opened this issue Dec 21, 2023 · 10 comments · Fixed by #2206
Assignees
Labels
bug 这个是一个 bug good first issue Good for newcomers 🌟 star

Comments

@zyl-fz
Copy link

zyl-fz commented Dec 21, 2023

问题描述
使用了一个市级行政区较大的Polygon Geojson绘制线图层后,重新调用style或者color方法就会出现该报错

平台
操作系统: [Windows]
网页浏览器: [Google Chrome]

屏幕截图或视频(可选)
image

补充说明(可选)

Copy link
Contributor

hi @zyl-fz, welcome!

Copy link
Contributor

Hi @zyl-fz, Please star this repo if you find it useful! Thanks ⭐!
你好 @zyl-fz。如果该仓库对你有用,可以 star 一下,感谢你的 ⭐!

@lzxue
Copy link
Contributor

lzxue commented Dec 21, 2023

什么情况下的保定错误,这个是 node 内存超出

@zyl-fz
Copy link
Author

zyl-fz commented Dec 21, 2023

什么情况下的保定错误,这个是 node 内存超出

使用了一个市级行政区较大的Polygon Geojson绘制线图层后,调用style或者color方法就会出现

@wlfei0502
Copy link

我也出现同样的问题,我在codesandbox上写了个demo:
https://codesandbox.io/p/sandbox/falling-fog-jqkc6s?file=%2Findex.js%3A49%2C12

@lzxue
Copy link
Contributor

lzxue commented Dec 21, 2023

感谢反馈,颜色更新报错

@lzxue lzxue added the bug 这个是一个 bug label Dec 21, 2023
@lzxue
Copy link
Contributor

lzxue commented Dec 21, 2023

是不是只有数据量大的时候出问题?本地验证小数据没有问题

@wlfei0502
Copy link

是的,小数据量没问题,我提供的数据就可以复现

@wlfei0502
Copy link

感觉是面数据的点太多,造成递归栈溢出

@lzxue lzxue linked a pull request Dec 21, 2023 that will close this issue
21 tasks
@zyl-fz
Copy link
Author

zyl-fz commented Dec 21, 2023

是不是只有数据量大的时候出问题?本地验证小数据没有问题

对,只有数据点很多的时候才会出现

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 这个是一个 bug good first issue Good for newcomers 🌟 star
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants