Open andrea-pasquale opened 1 week ago
@andrea-pasquale I'm sorry for the tuple business. As soon as I will finish #909 I have other few simpler improvements in mind, including a consistent migration to Pydantic. Whenever that will be completed, tuples will be available again, since the serialization will come out of the box.
Unfortunately, I will have to balance those with #869 (but if I will be good enough with #909, a first version of that should be pretty simple to achieve).
@andrea-pasquale I'm sorry for the tuple business. As soon as I will finish #909 I have other few simpler improvements in mind, including a consistent migration to Pydantic. Whenever that will be completed, tuples will be available again, since the serialization will come out of the box.
Unfortunately, I will have to balance those with #869 (but if I will be good enough with #909, a first version of that should be pretty simple to achieve).
No problem @alecandido. Now I believe that it should be fixed. I will try to take care of tests now. In which order do you recommend that we merge this alongside #909 and #869?
In which order do you recommend that we merge this alongside #909 and #869?
Do not bother about #909 and #869: whenever you're ready, we review and merge. If it comes before the others, I will rebase myself ( ╥ ᴗ ╥)
Attention: Patch coverage is 98.87640%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 97.45%. Comparing base (
4b8d407
) to head (9da5af8
). Report is 2 commits behind head on main.
Now it should be ready to review. There are a lot of file changes just for the tuple issue
Addition of a
qq update
program to generate the updated platform from CLI after the qibocal execution. Closes #833.Checklist:
master
main
main