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

Cannot console.log objects having circular references #3178

Open
dipamsen opened this issue Jun 30, 2024 · 10 comments
Open

Cannot console.log objects having circular references #3178

dipamsen opened this issue Jun 30, 2024 · 10 comments

Comments

@dipamsen
Copy link

dipamsen commented Jun 30, 2024

p5.js version

1.9.4

What is your operating system?

Windows

Web browser and version

Chrome 126.0.6478.127

Actual Behavior

Printing a p5.Image object breaks the console.

Expected Behavior

p5.Image should be printable.

Steps to reproduce

Steps:

  1. Copy the following code in a sketch
  2. Running the code prints nothing on the console
  3. Commenting out the third line prints "Hi".

Snippet:

let img = createImage(20, 20);
console.log("Hi");
console.log(img);
@dipamsen dipamsen added the Bug label Jun 30, 2024
@dipamsen dipamsen changed the title Cannot print p5.Image Cannot console.log p5.Image Jun 30, 2024
@dipamsen dipamsen changed the title Cannot console.log p5.Image Cannot console.log p5.Image object Jun 30, 2024
@dipamsen
Copy link
Author

dipamsen commented Jul 1, 2024

More information: The source of this issue seems to be that recursive data structures cannot be logged in the console. (This limitation is not present in the latest version of console-feed)

Example:

let obj = {};
obj.a = obj;
console.log(obj); // breaks console, "maximum call stack exceeded"

Other objects which have a recursive structure and thus break the console:

  • All custom p5 class instances, as they have a obj._pInst.renderer._pInst. ... recursive definition
  • Window / globalThis
  • Any object/class instance which stores one of the above in a field

@dipamsen dipamsen changed the title Cannot console.log p5.Image object Cannot console.log objects having circular references Jul 1, 2024
@raclim
Copy link
Collaborator

raclim commented Jul 3, 2024

Thanks for raising this and looking into it!

@geekaryan
Copy link

@dipamsen hello there, i just looked into this issue. I am trying to understand it more in depth to came out with a solution. I want to know what expected behavior we want when we encounter this type of circular loop data structure ? What output should be logged out so user know he has done a mistake.

@dipamsen
Copy link
Author

dipamsen commented Oct 18, 2024

@geekaryan I dont think it should be interpreted as a mistake. The library which is being used to render the "console" on the web editor (console-feed) is perfectly capable of logging such objects. (All major browsers also of course support logging of these objects).

The error we are hitting seems to be from immer/redux (?) which is unable to serialise these infinitely recursive objects. (not sure if this is correct).

@geekaryan
Copy link

@dipamsen got your point let me try to run this locally on my system and try to regenerate this issue. Thank you

@jht9629-nyu
Copy link

Similar failure trying to display image variables in the console entry area from this sketch:
https://editor.p5js.org/jht9629-nyu/sketches/N1xP0mzu-
Enter img1 in the console entry area and you get this error in the browser console:
Uncaught RangeError: Maximum call stack size exceeded
at immer.esm.mjs:1:912
at Array.forEach ()
at ht (immer.esm.mjs:1:904)
at Bt (immer.esm.mjs:1:2716)
at It (immer.esm.mjs:1:3259)
at immer.esm.mjs:1:2742
at immer.esm.mjs:1:948
at Array.forEach ()
at ht (immer.esm.mjs:1:904)
at Bt (immer.esm.mjs:1:2716

@adwaitanand
Copy link

Hi @dipamsen I have been working on this issue for the past few days i have attached a screenshot
P5 Console Log issue Fix
please let me know is this the expected behavior

@dipamsen
Copy link
Author

@adwaitanand Thats pretty good, however I am pretty sure earlier it was possible to actually print circular structures in the p5 console, i.e. you could infinitely click the _pixelsState property which would render the recursive subtree incrementally. This feature is indeed supported by console-feed.

As I understand it, the problem is in serialisation/deserialisation, so if it is not possible to recover the object in its entirity, then perhaps this is the next best option. I'll leave it to the project maintainers to decide.

@adwaitanand
Copy link

Thank you @dipamsen . Initially, I attempted to resolve the issue by allowing infinite clicks on the _pixelsState property, but I kept encountering the error: "Immer forbids circular reference." This led me to replace the circular reference object with a string.

P5 Immer Error

However, I’ll explore whether it’s possible to render the circular reference object directly in the console feed.

@adwaitanand
Copy link

@dipamsen and @raclim I found a way to render the objects for circular reference .

P5 Solution for Console Issue

I'm new to open source and need help with the PR for this issue. When I run npm run test in the Docker shell, some tests fail, for files I didn’t modify. I tried cloning a fresh project and running the tests without making any changes, but the same tests failed .

Can someone please guide me on what I might be doing wrong or the correct steps to proceed?
test error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants