qubit count update

[ZachMassa]

Updating the H2-1 target to 52 qubits to match current performance specifications.

[kikomiss]

@ZachMassa LGTM, I think we also would need to update number of qubits for h2-1 here

azure-quantum-python/azure-quantum/azure/quantum/qiskit/backends/quantinuum.py

Line 62 in 778a089

return 32

[ZachMassa]

@ZachMassa LGTM, I think we also would need to update number of qubits for h2-1 here

azure-quantum-python/azure-quantum/azure/quantum/qiskit/backends/quantinuum.py

Line 62 in 778a089

return 32

Thank you for catching this. It should be updated now.

[kikomiss]

Sorry, just found why the unit-tests are failing. We're also going to need to update this test:

azure-quantum-python/azure-quantum/tests/unit/test_qiskit.py

Line 1176 in 7f96bb5

def test_configuration_quantinuum_backends(self):

it assumes that h2-1 uses 32 qubits

[xinyi-joffre]

@ZachMassa, following up on this PR to see if you still wanted to merge this? I think there's the remaining failing unit test that @kikomiss pointed out, and then we can merge.

[ZachMassa]

@ZachMassa, following up on this PR to see if you still wanted to merge this? I think there's the remaining failing unit test that @kikomiss pointed out, and then we can merge.

I am fine with merging it. It seems like the issue mentioned above is that the unit test assumed H2 has a qubit count of 32 instead of 56.

[xinyi-joffre]

@ZachMassa , can you make the fix for the unit test, since these changes are on your fork? Otherwise, the pipelines won't pass in order for your PR to merge.

[ZachMassa]

@ZachMassa , can you make the fix for the unit test, since these changes are on your fork? Otherwise, the pipelines won't pass in order for your PR to merge.

@xinyi-joffre I believe that these tests should be updated to reflect the new qubit count now.

[kikomiss]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[kikomiss]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[xinyi-joffre]

Thank you for the update, @ZachMassa ! It's been merged!