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

Deployment gas computation for EraVM #675

Open
Karrq opened this issue Oct 29, 2024 · 0 comments
Open

Deployment gas computation for EraVM #675

Karrq opened this issue Oct 29, 2024 · 0 comments
Assignees
@Karrq @elfedy
Labels
needs more info ℹ️ Item requires more detail p2 🟡 Indicates moderately high priority item upstream-parity 🟰 Needed for upstream feature parity

Comments

@Karrq
Copy link
Contributor

Karrq commented Oct 29, 2024

Improve gas limit fix computation, to detect if the passed in gas is sufficient for the full deployment

This would be easier to troubleshoot if the EraVM failed with a more useful error like “out of gas”

Might be outdated at this point, due to recent changes to gas values:
@Karrq Karrq added question❓ Further information is requested p2 🟡 Indicates moderately high priority item upstream-parity 🟰 Needed for upstream feature parity needs more info ℹ️ Item requires more detail and removed question❓ Further information is requested labels Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Karrq Karrq

@elfedy elfedy

Labels
needs more info ℹ️ Item requires more detail p2 🟡 Indicates moderately high priority item upstream-parity 🟰 Needed for upstream feature parity
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Karrq @elfedy