pasqal-io / Pulser

Library for pulse-level/analog control of neutral atom devices. Emulator with QuTiP.
Apache License 2.0
159 stars 57 forks source link

Remove Chadoq2 device #607

Closed madagra closed 7 months ago

madagra commented 7 months ago

The Chadoq2 device is currently deprecated and it should be removed.

HGSilveri commented 7 months ago

Rather than removing Chadoq2 altogether - which would be quite disruptive to existing code - I propose the following steps:

@madagra @a-corni Do you agree with these steps? Would a different name perhaps be better? I also thought of HybridDevice which is shorter but vaguer

a-corni commented 7 months ago

I agree with these steps ! I prefer DigitalAnalogDevice to HybridDevice, I think it's more explicit.

madagra commented 7 months ago

Let's go ahead with DigitalAnalogDevice.