The test failure here in CI isn't unrelated: at the moment there's a place in the code where we accidentally call max([]), which is undefined because if there's a sequence of zero elements, there's nothing that can be the maximum. I didn't look deeply, but there's likely a place where we should have short-circuited out of the method before trying to call max.

Thank you! It was a simply fix by reordering the checks for edge cases.

I looked at the logs for Run Docs build and am confused where the issue arises from. Not sure how to proceed.

The docs failing wasn't your fault - just a dodgy connection from the VM to crates.io. I've re-run it, because it should just be ephemeral.

Thank you for opening a new pull request.

Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient.

While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone.

One or more of the the following people are requested to review this:

• @Qiskit/terra-core
• @ikkoham