Qiskit / qiskit-ibm-runtime / 27195034706 / 5
87%
main: 87%

DEFAULT BRANCH: main
Ran
Files 313
Run time 13s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 86.936% (+0.7%) from 86.249%
27195034706.5

push

github

web-flow 
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.

7932 of 9124 relevant lines covered (86.94%)

0.87 hits per line

Source Files on job unit-tests_python3.11-ubuntu-latest - 27195034706.5