Christian Fritz

Hi @baalexander, I've taken it on to complete the work you started -- thanks for getting all the hard stuff done already! My repo is https://github.com/chfritz/rosnodejs. So far I fixed...

oh wow, I had no idea rosjs existed. I guess this is very recent development, too. Thanks for mentioning this. I'll have to try that out. So far all I...

I've seen this problem before. It's because your ROS_PACKAGE_PATH environment variable contains this path but it doesn't usually exist. You can try running with: ``` env ROS_PACKAGE_PATH="/opt/ros/jade/share nodejs app.js ```

I think this is due to `observedAttributes` missing in this package. See https://developer.mozilla.org/en-US/docs/Web/Web_Components/Using_custom_elements#using_the_lifecycle_callbacks. Someone has already proposed a fix, but it wasn't merged: https://github.com/LukasBombach/react-web-component/pull/36.

@panshengjie thanks for reporting. Do you have a canonical code example to reproduce this? That would help us fix this more promptly.

Is this a duplicate of https://github.com/RethinkRobotics-opensource/rosnodejs/issues/117?

@trusktr , I'm trying to understand. You are saying that the *browser* does a major GC. But rosnodejs is a server-side package only. Are we really talking about the same...

Would it make sense to wait and see how ROS core decides to resolve this issue first? I think it would be very confusing to use different terminology in the...

No, I haven't spoken with anyone at OSRF lately. I agree though that is could end up being a huge change. It might even require every ROS user to update...

I've discovered that `npm install [email protected]` works, so it seems that the issue was introduced recently.