Hi @javabster . I think that is because the default value of vertical compression in _text_circuit_drawer() method is high. I guess that's why there's an overlap in the H gate. However, the value of vertical compression in the default .draw() method is medium. For instance the same test circuit when drawn using .draw() method gives me an output as below

Ok it's good that it isn't a bug in .draw() but I think it should still be fixed for the _text_circuit_drawer(), perhaps by adjusting the vertical compression for cases like this

Hi @javabster . I think that is because the default value of vertical compression in _text_circuit_drawer() method is high. I guess that's why there's an overlap in the H gate. However, the value of vertical compression in the default .draw() method is medium. For instance the same test circuit when drawn using .draw() method gives me an output as below

Ok it's good that it isn't a bug in .draw() but I think it should still be fixed for the _text_circuit_drawer(), perhaps by adjusting the vertical compression for cases like this

If I am understanding correct, should we change the value of vertical_compression of text_circuit_drawer() to medium for these tests? Or would it be better if we have tests for both medium and default vertical compressions in the tests?

Hi @javabster . I think that is because the default value of vertical compression in _text_circuit_drawer() method is high. I guess that's why there's an overlap in the H gate. However, the value of vertical compression in the default .draw() method is medium. For instance the same test circuit when drawn using .draw() method gives me an output as below

Ok it's good that it isn't a bug in .draw() but I think it should still be fixed for the _text_circuit_drawer(), perhaps by adjusting the vertical compression for cases like this

If I am understanding correct, should we change the value of vertical_compression of text_circuit_drawer() to medium for these tests? Or would it be better if we have tests for both medium and default vertical compressions in the tests?

yep @TharrmashasthaPV that's correct!

@1ucian0 any thoughts about using _text_circuit_drawer() instead of .draw()`?

@1ucian0 any thoughts about using _text_circuit_drawer() instead of .draw()?

In other tests we do the same. I think is good for now. At same point probably makes sense to refactor them at some point.

After #6370, it seems that we are moving towards bullet representations when they are possible.

Wouldn't be better something like this?

qr = QuantumRegister(2, "qr")
cr = ClassicalRegister(2, "cr")
circuit = QuantumCircuit(qr, cr)
circuit.h(qr[0]).c_if(cr[0], 1)
circuit.h(qr[1]).c_if(cr[1], 0)
circuit.draw('text', cregbundle=False)

       ┌───┐        
qr_0: ─┤ H ├────────
       └─╥─┘ ┌───┐ 
qr_1: ───╫───┤ H ├─
         ║   └─╥─┘ 
cr_0: ═══■═════╬═══
               ║
cr_1: ═════════o═══

(may understanding of your code is that this change would be easy to make. Please, ignore my comment otherwise)

After #6370, it seems that we are moving towards bullet representations when they are possible.

Wouldn't be better something like this?

qr = QuantumRegister(2, "qr")
cr = ClassicalRegister(2, "cr")
circuit = QuantumCircuit(qr, cr)
circuit.h(qr[0]).c_if(cr[0], 1)
circuit.h(qr[1]).c_if(cr[1], 0)
circuit.draw('text', cregbundle=False)

       ┌───┐        
qr_0: ─┤ H ├────────
       └─╥─┘ ┌───┐ 
qr_1: ───╫───┤ H ├─
         ║   └─╥─┘ 
cr_0: ═══■═════╬═══
               ║
cr_1: ═════════o═══

(may understanding of your code is that this change would be easy to make. Please, ignore my comment otherwise)

Thanks for pointing this out. I have made this change in the latest commit. @javabster I have also made the change in the vertical_compression value in the test.