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(ai): fix cli prices nil error #3063

Merged
merged 1 commit into from
May 20, 2024
Merged

fix(ai): fix cli prices nil error #3063

merged 1 commit into from
May 20, 2024

Conversation

rickstaa
Copy link
Member

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

This commit ensures that the livepeer_cli does not throw a nil error when it tries to retrieve the orchestrator base price.

Specific updates (required)

  • Ensures that the CLI can handle nil price being returned.

How did you test each of these updates (required)

  • Ran the client.

Does this pull request close any open issues?

Checklist:

@github-actions github-actions bot added the AI Issues and PR related to the AI-video branch. label May 20, 2024
@eliteprox
Copy link
Collaborator

Tested and this is working

Copy link
Collaborator

@eliteprox eliteprox left a comment

Choose a reason for hiding this comment

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

Tested and this is working. Later on we can add a way to list the model prices on livepeer_cli

This commit ensures that the livepeer_cli does not throw a `nil` error
when it tries to retrieve the orchestrator base price.
@rickstaa rickstaa merged commit b7a9fb7 into ai-video May 20, 2024
7 of 9 checks passed
@rickstaa rickstaa deleted the fix_livepeer_cli branch May 20, 2024 22:54
@rickstaa rickstaa mentioned this pull request Jun 10, 2024
5 tasks
eliteprox pushed a commit to eliteprox/go-livepeer that referenced this pull request Jul 26, 2024
This commit ensures that the livepeer_cli does not throw a `nil` error
when it tries to retrieve the orchestrator base price.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AI Issues and PR related to the AI-video branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants