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

Multiple Realms being created? #169

Closed
katelynsills opened this issue May 16, 2019 · 2 comments
Closed

Multiple Realms being created? #169

katelynsills opened this issue May 16, 2019 · 2 comments
Labels
SwingSet package: SwingSet

Comments

@katelynsills
Copy link
Contributor

I'm in the process of reorganizing the kernel state, and moved the kernel state out of kernel.js and into another file under another sub-directory. This included state related functions, like dump and getState and loadState. To create the kernel obj in kernel.js, I just changed the properties of the kernel obj, such as dump: kernelState.dump, where kernelState.dump is an import from the other file.

I was getting an error: (node:82523) UnhandledPromiseRejectionWarning: TypeError: prototype function () { [native code] } of unknown.dump is not already in the fringeSet

I asked Mark about it and he suggested trying

dump() {
  return kernelState.dump();
},

And that worked. Mark says this is a symptom of having multiple realms. Also, the prototype of dump was not Function.prototype and had an arguments property that Function.prototype didn't have.

@katelynsills
Copy link
Contributor Author

Ah! I figured it out, I wasn't making the kernelState inside the stringified code. False alarm.

@warner
Copy link
Member

warner commented Dec 4, 2019

in the old repo. this was SwingSet issue 47

@warner warner transferred this issue from Agoric/SwingSet Dec 4, 2019
@warner warner added the SwingSet package: SwingSet label Dec 4, 2019
dckc pushed a commit to dckc/agoric-sdk that referenced this issue Dec 5, 2019
deps: upgrade to working eventual-send
dckc pushed a commit to dckc/agoric-sdk that referenced this issue Dec 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SwingSet package: SwingSet
Projects
None yet
Development

No branches or pull requests

2 participants