search

PolymathNetwork / polymath.js

10 stars 6 forks source link

Estimate gas #4

Closed bshevchenko closed 6 years ago

bshevchenko commented 6 years ago

for SecurityTokenRegistry.generateSecurityToken works only with 5% increase for whitelist remove entry works only with 100% increase (see #6)

bshevchenko commented 6 years ago

Probably we can just catch OOG errors and increase gas limit until it'll be okay (with maximum 2 or 3 iterations of recursion) See possible errors messages in #6

bshevchenko commented 6 years ago

Contract._tx now has gasLimit param, which accepts gas limit multiplier or exact amount of gas. So we have selectively adjusted estimation for necessary cases.