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

Support standalone serialization of `Register` and `RegisterLayout` to the abstract representation + Hashing for coordinate collections #627

Closed HGSilveri closed 6 months ago

HGSilveri commented 6 months ago
a-corni commented 6 months ago

I am wondering if the serialization of the DetuningMap will not be needed as well...

HGSilveri commented 6 months ago

I am wondering if the serialization of the DetuningMap will not be needed as well...

That's a good point, I guess it might be at some point. We can either do it right now or wait until it is needed. I don't have a strong preference, I can find pros and cons for both options tbh

a-corni commented 6 months ago

I am wondering if the serialization of the DetuningMap will not be needed as well...

That's a good point, I guess it might be at some point. We can either do it right now or wait until it is needed. I don't have a strong preference, I can find pros and cons for both options tbh

Let's wait until it is needed then, the current PR is already finished and local addressability is perhaps not stable.