Tarun Kumar Allamsetty
Tarun Kumar Allamsetty
I am not sure how to [fix failing CI check](https://github.com/PennyLaneAI/pennylane/actions/runs/8039250831/job/21956066233?pr=5256), Can the reviewers please guide me?
@trbromley Thank you for your response. If there's anything I can do to help with the PennyLane 0.35 release preparations, please let me know. I'd be more than happy to...
@timmysilv all CI checks passed 🎉 , thanks for the fix. This PR is now ready for review.
Hey @albi3ro, I noticed that all classes which inherit from SampleMeasurement implement process_counts except SampleMP. Should I add an implementation for that?
Hi @albi3ro and @trbromley, Thanks for reviewing my PR. It's my first open-source contribution, and I'm eager to contribute even more 🎉. Appreciate it!
@albi3ro, regarding adding `SampleMP.process_counts`. Should I create a PR directly or make an issue about it? What's the preferred process?
@tanujkhattar , can I work on this issue? I am new to open source, in case of any queries where can I reach out for help ?
@95-martin-orion and @tanujkhattar , the constructors in [pauli_measurement_gates.py](https://github.com/quantumlib/Cirq/blob/master/cirq-core/cirq/ops/pauli_measurement_gate.py) don't take confusion map as parameter, but they are taken in [measurement_gate.py](https://github.com/quantumlib/Cirq/blob/master/cirq-core/cirq/ops/measurement_gate.py). So is this issue still open ?
Great, I will start looking into this Thank you @95-martin-orion !!
@Kirs67 I'm working on this issue