10374261794
89%

Ran 13 Aug 2024 05:43PM UTC

Jobs 1

Files 861

Run time 2min

Badge

Embed ▾

Committed 13 Aug 2024 04:41PM UTC coverage: 89.676% (-0.03%) from 89.705%

Build # 10374261794

Build Type

push

github

Committed by

web-flow

Commit Message

Use hashbrown::HashSet instead of ahash::HashSet (#12951)

In the `target_transpiler/mod.rs` module we were using ahash::HashSet
for a hash set implementation, but the rest of Qiskit has standardized
on using hashbrown for the `HashSet` and `HashMap` types. Hashbrown uses
ahash for it's hashing algorithm but it also provides other advantages.
To ensure that hash sets are compatible across the library we should be
using the same library for everything. To support this goal, this commit
also adds a clippy rule that raises a warning if the std library hashmap
or hashset is used, or the versions from ahash. This means with our
current dependency set the only allowed hashset types are
`hashbrown::HashMap`/`HashSet` and `indexmap::IndexMap`/`IndexSet` (for
where we need to maintain insertion order). Ideally we'd have a rule
that forces the use of ahash with `IndexMap` and `IndexSet` (see #12935)
but I don't think clippy exposes an option to enable something like that.

Run Details

67404 of 75164 relevant lines covered (89.68%)

386436.5 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
2	88.39	-0.36%	qiskit/transpiler/passes/synthesis/unitary_synthesis.py
5	91.73	-0.5%	crates/qasm2/src/lex.rs
6	90.84	-4.58%	qiskit/synthesis/two_qubit/xx_decompose/decomposer.py
12	97.15	-0.92%	crates/qasm2/src/parse.rs

Jobs

ID	Job ID	Ran	Files	Coverage
1	10374261794.1	13 Aug 2024 05:43PM UTC	861	89.68	GitHub Action Run

Qiskit / qiskit / 10374261794
89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 10374261794

Qiskit / qiskit / 10374261794 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 10374261794

Qiskit / qiskit / 10374261794
89%

README BADGES
x