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

Gas usage profiling #46

Closed
Dentosal opened this issue Nov 8, 2021 · 0 comments · Fixed by #47
Closed

Gas usage profiling #46

Dentosal opened this issue Nov 8, 2021 · 0 comments · Fixed by #47
Assignees
@Dentosal
Labels
enhancement New feature or request fuel-vm Related to the `fuel-vm` crate.

Comments

@Dentosal
Copy link
Member

Dentosal commented Nov 8, 2021 •
edited
Loading

Per instruction gas usage profiling.

  • Enabled using a feature flag
  • Can be used with source mapping to profile original source code
  • Attempt to use standard formats for output, maybe perf or pprof
@Dentosal Dentosal added the enhancement New feature or request label Nov 8, 2021
@Dentosal Dentosal self-assigned this Nov 8, 2021
@Voxelot Voxelot mentioned this issue Nov 9, 2021
Closed
@Dentosal Dentosal mentioned this issue Nov 15, 2021
Merged
ControlCplusControlV pushed a commit that referenced this issue Nov 24, 2022
@vlopes11
Replace ScriptResultRepr by PanicReason (#46)
cae1912 
The specs refer to what is implemented as script result representation
as `PanicReason`.

This panic reason can be used also outside script execution (Predicate
verification?) since the instructions execution will be the same for
both.

Using `ScriptResultRepr` for predicate verification is misleading.
ControlCplusControlV pushed a commit that referenced this issue Nov 24, 2022
@adlerjohn
Bump fuel-types to v0.3 (#46)
606176c
ControlCplusControlV pushed a commit that referenced this issue Nov 24, 2022
@xgreenx
Added const for base asset (#46)
fb159ff 
* Added const for base asset

* Move `BASE` to `AssetId`
ControlCplusControlV pushed a commit that referenced this issue Dec 1, 2022
@vlopes11
Replace ScriptResultRepr by PanicReason (#46)
b9752c3 
The specs refer to what is implemented as script result representation
as `PanicReason`.

This panic reason can be used also outside script execution (Predicate
verification?) since the instructions execution will be the same for
both.

Using `ScriptResultRepr` for predicate verification is misleading.
ControlCplusControlV pushed a commit that referenced this issue Dec 1, 2022
@adlerjohn
Bump fuel-types to v0.3 (#46)
5156f1f
ControlCplusControlV pushed a commit that referenced this issue Dec 1, 2022
@xgreenx
Added const for base asset (#46)
3ace4b0 
* Added const for base asset

* Move `BASE` to `AssetId`
@mitchmindtree mitchmindtree added the fuel-vm Related to the `fuel-vm` crate. label Dec 9, 2022
xgreenx pushed a commit that referenced this issue Dec 20, 2022
@vlopes11 @xgreenx
Replace ScriptResultRepr by PanicReason (#46)
4ec9d8e 
The specs refer to what is implemented as script result representation
as `PanicReason`.

This panic reason can be used also outside script execution (Predicate
verification?) since the instructions execution will be the same for
both.

Using `ScriptResultRepr` for predicate verification is misleading.
xgreenx added a commit that referenced this issue Dec 20, 2022
@xgreenx
Added const for base asset (#46)
cac2ce1 
* Added const for base asset

* Move `BASE` to `AssetId`
xgreenx pushed a commit that referenced this issue Dec 20, 2022
@adlerjohn @xgreenx
Bump fuel-types to v0.3 (#46)
b6495d6
xgreenx pushed a commit that referenced this issue Dec 20, 2022
@vlopes11 @xgreenx
Replace ScriptResultRepr by PanicReason (#46)
6adc998 
The specs refer to what is implemented as script result representation
as `PanicReason`.

This panic reason can be used also outside script execution (Predicate
verification?) since the instructions execution will be the same for
both.

Using `ScriptResultRepr` for predicate verification is misleading.
xgreenx added a commit that referenced this issue Dec 20, 2022
@xgreenx
Added const for base asset (#46)
47c881d 
* Added const for base asset

* Move `BASE` to `AssetId`
xgreenx pushed a commit that referenced this issue Dec 20, 2022
@adlerjohn @xgreenx
Bump fuel-types to v0.3 (#46)
11921c7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Dentosal Dentosal

Labels
enhancement New feature or request fuel-vm Related to the `fuel-vm` crate.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add gas profiling FuelLabs/fuel-vm
2 participants
@Dentosal @mitchmindtree