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

feat: support additional compute budget ixs #25104

Merged
merged 1 commit into from
May 20, 2022

Conversation

jstarry
Copy link
Member

@jstarry jstarry commented May 10, 2022

Problem

The compute budget interface will be updated before being activated on mainnet and the web3 api needs to be updated to match. For more details see: #25178

Summary of Changes

  • Added support for the new SetComputeUnitLimit and SetComputeUnitPrice prioritization fee ixs

Fixes #

@codecov
Copy link

codecov bot commented May 10, 2022

Codecov Report

Merging #25104 (e6ae3b1) into master (be1d606) will decrease coverage by 7.0%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #25104       +/-   ##
===========================================
- Coverage    82.1%    75.1%     -7.1%     
===========================================
  Files         616       38      -578     
  Lines      169469     2314   -167155     
  Branches        0      335      +335     
===========================================
- Hits       139161     1738   -137423     
+ Misses      30308      459    -29849     
- Partials        0      117      +117     

@jstarry
Copy link
Member Author

jstarry commented May 10, 2022

This change is blocked until the runtime change is merged and released in a new version of solana-test-validator

@jstarry jstarry added the blocked Unable to proceed label May 10, 2022
@jstarry jstarry force-pushed the web3/compute-budget branch from 0736ea2 to 536aa65 Compare May 20, 2022 07:16
@jstarry jstarry removed the blocked Unable to proceed label May 20, 2022
@jstarry jstarry force-pushed the web3/compute-budget branch from 536aa65 to e6ae3b1 Compare May 20, 2022 07:21
@jstarry jstarry added the automerge Merge this Pull Request automatically once CI passes label May 20, 2022
@jstarry jstarry changed the title feat: migrate to updated compute budget program interface feat: support additional compute budget ixs May 20, 2022
@mergify mergify bot merged commit a7e6f4d into solana-labs:master May 20, 2022
@jstarry jstarry deleted the web3/compute-budget branch May 20, 2022 07:28
jeffwashington pushed a commit to jeffwashington/solana that referenced this pull request Jun 29, 2022
jeffwashington pushed a commit to jeffwashington/solana that referenced this pull request Jun 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant