Skip to content

docs: add instructions for setting additional fees during network congestion#90

Open
rishi-jat wants to merge 2 commits into
SuperteamDAO:masterfrom
rishi-jat:fix/issue/88
Open

docs: add instructions for setting additional fees during network congestion#90
rishi-jat wants to merge 2 commits into
SuperteamDAO:masterfrom
rishi-jat:fix/issue/88

Conversation

@rishi-jat

@rishi-jat rishi-jat commented Jun 21, 2025

Copy link
Copy Markdown

The ComputeBudgetProgram class is now implemented as requested.

  • Both set_compute_unit_limit and set_compute_unit_price methods have been added.
  • Tests for both methods are included in tests/test_compute_budget.py.
  • Documentation and usage examples are now accurate and match the implementation.

close:#88

Let me know if you need any further changes!

@GitBolt

GitBolt commented Jun 21, 2025

Copy link
Copy Markdown
Contributor

Hi, this is not correct because Solathon does not have ComputeBudgetProgram class anywhere. The purpose of the issue was to introduce this class for people to be able to utilize it. You just added documentation for something that doesn't exist yet.

@rishi-jat

Copy link
Copy Markdown
Author

Hi! Sorry about that I misunderstood the purpose of the issue. I now realize that the ComputeBudgetProgram class doesn’t exist yet and the goal is to implement it, not just add documentation for it.

@rishi-jat

Copy link
Copy Markdown
Author

I have now implemented the ComputeBudgetProgram class in the core library as requested, including both set_compute_unit_limit and set_compute_unit_price methods. Unit tests have also been added to cover these new methods. The documentation and usage examples now accurately reflect the current implementation.

Please take a look at the updated PR when you have a chance. Let me know if any further changes are needed! make this short and simple

@rishi-jat

Copy link
Copy Markdown
Author

Hi @GitBolt, could you please review my PR whenever you get a chance?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants