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

[important] Changes in populateIndexes required soon! #34

Closed
juliancwirko opened this issue Feb 14, 2022 · 1 comment
Closed

[important] Changes in populateIndexes required soon! #34

juliancwirko opened this issue Feb 14, 2022 · 1 comment
Labels
enhancement New feature or request idea v1

Comments

@juliancwirko
Copy link
Member

https://stackoverflow.com/questions/71108759/vecmapper-extend-from-slice-differences-between-testnet-and-devnet

There are some limitations regarding Rust's dynamic allocation, which can cause some sc calls to fail. We recommend using managed types instead, which only allocate memory inside the VM, or use static buffers.

The simplest and fastest solution is to use only VecMapper and 'push', which will consume over twice as much gas. As a temporary solution, it should be a quick fix.

@juliancwirko juliancwirko added enhancement New feature or request idea v1 labels Feb 14, 2022
This was referenced Feb 14, 2022
@juliancwirko
Copy link
Member Author

After tests on the testnet it looks like it could be 3000 instead of the current 2200, but it can be implemented when devnet and testnet have equal codebase and configuration. For now, the limit per transaction is 2200 indexes to populate. CLI will do this automatically in v1.4.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request idea v1
Projects
None yet
Development

No branches or pull requests

1 participant