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

Support new encoder/decoder for scripts and predicates #1279

Closed
digorithm opened this issue Feb 12, 2024 · 3 comments
Closed

Support new encoder/decoder for scripts and predicates #1279

digorithm opened this issue Feb 12, 2024 · 3 comments
Assignees
@hal3e
Labels
epic An epic is a high-level master issue for large pieces of work.

Comments

@digorithm
Copy link
Member

Part of #1246.

Blocked by:
@digorithm digorithm mentioned this issue Feb 12, 2024
Closed
@kamyar-tm kamyar-tm added the epic An epic is a high-level master issue for large pieces of work. label Feb 15, 2024
@Matttioboy
Copy link

epic work

@hrrl
Copy link

hrrl commented Mar 11, 2024

good

@hal3e hal3e mentioned this issue Mar 21, 2024
Merged
5 tasks
hal3e added a commit that referenced this issue Apr 2, 2024
@hal3e
feat: experimental encoding for contracts, scripts and predicates (#1303
0e99f41 
)

closes: #1278,
#1279,
#1046

This PR adds support for the new encoding scheme for contracts, scripts
and predicates.

I have added a new `ExperimentalBoundedEncoder` which can be activated
with the `experimental` cfg flag.
I have tried to minimize the impact of the new encoder as much as
possible to make it easier for review. A full refactor of the whole sdk
is necessary once the new encoding becomes the default one.

- The function selector changed and now it is the name of the method.
- The `CALL` opcode changed with the new encoding and is expecting the
following call data: ContractID, pointer to fn_selector (name of the
method), pointer to encoded arguments, number of coins, asset_id,
gas_forwarded.
@digorithm
Copy link
Member Author

Closing this because it's been mostly done. String slices and RawSlice support for scripts will be added later.

ak-tech007 added a commit to ak-tech007/Rust-SDK that referenced this issue Dec 6, 2024
@ak-tech007
feat: experimental encoding for contracts, scripts and predicates (#1…
0da4015 
…303)

closes: FuelLabs/fuels-rs#1278,
FuelLabs/fuels-rs#1279,
FuelLabs/fuels-rs#1046

This PR adds support for the new encoding scheme for contracts, scripts
and predicates.

I have added a new `ExperimentalBoundedEncoder` which can be activated
with the `experimental` cfg flag.
I have tried to minimize the impact of the new encoder as much as
possible to make it easier for review. A full refactor of the whole sdk
is necessary once the new encoding becomes the default one.

- The function selector changed and now it is the name of the method.
- The `CALL` opcode changed with the new encoding and is expecting the
following call data: ContractID, pointer to fn_selector (name of the
method), pointer to encoded arguments, number of coins, asset_id,
gas_forwarded.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hal3e hal3e

Labels
epic An epic is a high-level master issue for large pieces of work.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@digorithm @hal3e @kamyar-tm @hrrl @Matttioboy