Add backend refresh() method #1955
Conversation
kt474
added
the
Changelog: New Feature
| instance=self._instance, | ||
| ): | ||
| self._configuration = config | ||
| self._get_properties(datetime=python_datetime.now()) |
There was a problem hiding this comment.
Looking at the code, _get_properties() and properties() seem to nearly identical, with the exception that the latter support refresh. I wonder if we can consolidate them and just pass refresh=True.
There was a problem hiding this comment.
Yeah I think we can consolidate the two methods - i'll do that when fixing #1956
There was a problem hiding this comment.
ok is the plan to use _get_properties(datetime) in this PR and then switch to using properties() after #1956 is done?
There was a problem hiding this comment.
Yeah I think it's easier that way
qiskit_ibm_runtime/ibm_backend.py
Outdated
| ): | ||
| self._configuration = config | ||
| self._get_properties(datetime=python_datetime.now()) | ||
| self._get_defaults() |
There was a problem hiding this comment.
Looks like _get_defaults also uses cached data if it already exists.
qiskit-ibm-runtime/qiskit_ibm_runtime/ibm_backend.py
Lines 251 to 253 in f8323dc
Summary
Details and comments
Fixes #