-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Using DOM.js library kills console.log #3386
Comments
@jht9629-nyu This issue could be due to a recursive function that doesn't have an appropriate exit condition. causing the call stack to overflow. I'll look into it further to identify the root cause and confirm if that's the issue. |
The core problem is why is stack overflow triggered in p5js editor and not in standalone environment? |
Seems to be same problem as #3178 |
I don't think so. There is no circular reference involved in this case. Just the presence of the library include of DOM.js causes simple console.log commands link this to fail: DOM.js is a library by lenincompres |
You are right, that issue I linked is not relevant. Upon further analysis I have found the actual issue and described it in the DOM.js issue. |
@dipamsen Thank you greatly for your detailed analysis and recommendation |
p5.js version
1.11.1
What is your operating system?
Mac OS
Web browser and version
133.0.6943.127 (Official Build) (arm64)
Actual Behavior
Expected Behavior
Steps to reproduce
Steps:
Snippet:
Just the include of lenincompres/DOM.js causes simple console.log fails, eg:
DOM.js is a library by lenincompres included in the index.html
<script src="https://cdn.jsdelivr.net/gh/lenincompres/DOM.js@latest/DOM.js"></script>
Issue is also logged on his repo for the library:
lenincompres/DOM.js - console.log failure issue
The text was updated successfully, but these errors were encountered: