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.

Sign up for GitHub

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

Jump to bottom

[x/programs] Remove the VM from the simulator #756

Open
richardpringle opened this issue Mar 5, 2024 · 4 comments
Open

[x/programs] Remove the VM from the simulator #756

richardpringle opened this issue Mar 5, 2024 · 4 comments
Labels
lifecycle/stale wasmlanche

Comments

@richardpringle
Copy link
Contributor

See @patrick-ogrady's comment here
#743 (comment)

I think there is a future where we could rip this out and just stub the host function to the WASM engine? If we did want to provide this "chain" feel, then it would make sense to keep it (but that may be better handled by an actual devnet instance of a VM with the WASM engine).
@richardpringle
Copy link
Contributor Author

richardpringle commented Mar 5, 2024 •
edited

@najeal, this could be another interesting issue if you're looking for something. It's blocked on

But they should make their way into main soon

@richardpringle richardpringle mentioned this issue Mar 5, 2024
Merged
@najeal
Copy link
Contributor

najeal commented Mar 6, 2024

@richardpringle I'm on it as soon as you merge the two PRs 👍

@richardpringle
Copy link
Contributor Author

@najeal, this one's no longer blocked!

@najeal najeal mentioned this issue Mar 19, 2024
Closed
@github-actions GitHub Actions
Copy link

This issue has become stale because it has been open 60 days with no activity. Adding the lifecycle/frozen label will exempt this issue from future lifecycle events.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
lifecycle/stale wasmlanche
Projects
Platform Engineering Group
Status: Backlog 🗄️
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove vm from simulator najeal/hypersdk
2 participants
@richardpringle @najeal