Skip to content

BYOC: fix payment ticket count calc#3650

Merged
ad-astra-video merged 8 commits intomasterfrom
av/fix-byoc-ticket-count-calc
Jul 21, 2025
Merged

BYOC: fix payment ticket count calc#3650
ad-astra-video merged 8 commits intomasterfrom
av/fix-byoc-ticket-count-calc

Conversation

@ad-astra-video
Copy link
Collaborator

@ad-astra-video ad-astra-video commented Jun 26, 2025

What does this pull request do? Explain your changes. (required)

Updates createPayment to accurately calculate ticket count.

Specific updates (required)

  • The slice created with make did not have a length as expected. Instead it has a capacity of the provided len(ticket.SenderParams)

How did you test each of these updates (required)

Tested locally using Agent SPE request example and observed the ticket count changed with each change in timeout_seconds in the request to accurately send 1 ticket per second of expected compute.

Does this pull request close any open issues?

Checklist:

@ad-astra-video ad-astra-video requested a review from pschroedl June 26, 2025 12:56
@github-actions github-actions bot added the go Pull requests that update Go code label Jun 26, 2025
@codecov
Copy link

codecov bot commented Jun 26, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 31.96943%. Comparing base (32516bc) to head (64597cb).
Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                 @@
##              master       #3650         +/-   ##
===================================================
+ Coverage   31.86089%   31.96943%   +0.10854%     
===================================================
  Files            156         156                 
  Lines          47359       47364          +5     
===================================================
+ Hits           15089       15142         +53     
+ Misses         31382       31328         -54     
- Partials         888         894          +6     
Files with missing lines Coverage Δ
server/job_rpc.go 30.02309% <100.00000%> (+6.56200%) ⬆️

... and 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 32516bc...64597cb. Read the comment docs.

Files with missing lines Coverage Δ
server/job_rpc.go 30.02309% <100.00000%> (+6.56200%) ⬆️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@pschroedl pschroedl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Nice tests ;)

@ad-astra-video ad-astra-video merged commit e52c907 into master Jul 21, 2025
21 of 22 checks passed
@ad-astra-video ad-astra-video deleted the av/fix-byoc-ticket-count-calc branch July 21, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants