Qiskit / qiskit-ibm-runtime / 27194711136

87%

main: 89%

Build Type

push

github

Committed by web-flow

Commit Message

Re-organize / clean parts of utils package (#2894)

<!--
⚠️ The pull request title should be short detailed and understandable
for all.
⚠️ Also, please add a release note file using reno if the change needs
to be
  documented in the release notes.
⚠️ If your pull request fixes an open issue, please link to the issue.

✅ I have added the tests to cover my changes.
✅ I have updated the documentation accordingly.
✅ I have read the CONTRIBUTING document.
-->

### Summary

Revise the `utils` package, aiming to reduce the imports traversing
through it from other packages and remove unused code, as well as
focusing on reducing the amount of functionality in the really diverse
`utils.utils`:
* remove unused members
* for functions used by just one other package, move them to such
package:
   * `account.utils`
   * `api.utils`
   * moving `backend_encoder.py` to `fake_provider`
* moved the logging functionality to its own module

### Details and comments

Related to #2749

This is not meant to tackle all the possible revamps of `utils`. After
this step, `utils` roughly contains:
* utils used by the backends (backend converter, backend decoder) -
prone to be combined in a module
* utils for validation (validations and most of the items in
`utils.utils`) - prone to be combined together
* utils for a well defined purpose (deprecation, logging, default
session) - prone to stay or bubble up in some cases
* smaller functions for time conversion and other functionality used in
several places - requires more evaluation
* `json.py` - prone to be promoted to top level, requires some finesse
as it is public

Import-wise, tackling `json.py` remains the biggest source of import
complexity, and moving it out would alleviate most of the issues for
keeping a well-behaved `utils` package.

### AI/LLM disclosure

- [x] I didn't use LLM tooling, or only used it privately.

Coverage Stats

75 of 113 new or added lines in 10 files covered. (66.37%)

33 existing lines in 3 files now uncovered.

7932 of 9124 relevant lines covered (86.94%)

9.5 hits per line