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

Many methods needlessly retrieve job inputs #1307

Closed
jyu00 opened this issue Jan 8, 2024 · 0 comments · Fixed by #1308
Closed

Many methods needlessly retrieve job inputs #1307

jyu00 opened this issue Jan 8, 2024 · 0 comments · Fixed by #1308
Assignees
Labels
bug Something isn't working priority: high High priority issue (must have for current release)
Milestone

Comments

@jyu00
Copy link
Collaborator

jyu00 commented Jan 8, 2024

Describe the bug

By default, job inputs (including circuits) are returned with the GET /jobs/{id} endpoint. Since inputs can be large, it has an exclude_params parameters to not return inputs.

Many methods in runtime_job still needlessly retrieve job inputs, including job.result(). This can cause the method call to exceed the 60s session interactive timeout, which in term causes the session to become deactivated.

Steps to reproduce

Expected behavior

Every call to job_get() should exclude params except the one from job.inputs.

Suggested solutions

Instead of relying on the caller to remember to specifying exclude_params=True, it might be better to just make that the default.

Additional Information

  • qiskit-ibm-runtime version: 0.17.0
  • Python version:
  • Operating system:
@jyu00 jyu00 added bug Something isn't working priority: high High priority issue (must have for current release) labels Jan 8, 2024
@kt474 kt474 self-assigned this Jan 8, 2024
@kt474 kt474 added this to the 0.18.0 milestone Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority: high High priority issue (must have for current release)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants