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

fix I2I latency score #3093

Merged
merged 1 commit into from
Jul 8, 2024
Merged

fix I2I latency score #3093

merged 1 commit into from
Jul 8, 2024

Conversation

rickstaa
Copy link
Member

@rickstaa rickstaa commented Jul 4, 2024

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

This pull request ensures that the number of requested output images are also taken into account for the I2I pipeline. This is already done for the T2I pipeline but this logic was missing in the I2I pipeline causing the latency score to be incorrect.

Specific updates (required)

  • Update the I2I latency score calculated by the gateway and used in the selection algorithm to include the number of requested output images.

How did you test each of these updates (required)

Run the code locally.

Does this pull request close any open issues?

AI-504

Checklist:

This commit ensures that the I2I pipeline latency score calculation now
considers the number of images.
@rickstaa rickstaa changed the base branch from master to ai-video July 4, 2024 14:39
@rickstaa rickstaa merged commit 29d4603 into ai-video Jul 8, 2024
1 check passed
@rickstaa rickstaa deleted the fix_I2I_latency_score branch July 8, 2024 12:00
eliteprox pushed a commit to eliteprox/go-livepeer that referenced this pull request Jul 26, 2024
…3093)

This commit ensures that the I2I pipeline latency score calculation now
considers the number of images.
eliteprox pushed a commit to eliteprox/go-livepeer that referenced this pull request Jul 26, 2024
…3093)

This commit ensures that the I2I pipeline latency score calculation now
considers the number of images.
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.

1 participant