[QHC-64]Compilation at platform level

[visagim]

• Fix coverage
• Changelog

Summary

• Compilation tasks moved from qblox_module, qblox_qrm, qblox_qcm to pulses.qblox_compiler
• Unified and removed some non-acquisition related settings from qrm / qcm under qblox_module
• Improved / added some of the qblox modules tests which were missing
• Removed bus related compiler methods

[linear]

QHC-64 Compiler at platform level

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (097bee6) 95.82% compared to head (382d1f0) 95.98%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #651      +/-   ##
==========================================
+ Coverage   95.82%   95.98%   +0.16%     
==========================================
  Files         270      271       +1     
  Lines        8815     8797      -18     
==========================================
- Hits         8447     8444       -3     
+ Misses        368      353      -15     

Flag	Coverage Δ
unittests	95.98% <100.00%> (+0.16%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[AlbertMitjans]

This is a very big PR changing core functionalities of qililab. I think we should make 2 people review and approve the PR before merging it.

[visagim]

This is a very big PR changing core functionalities of qililab. I think we should make 2 people review and approve the PR before merging it.

Yes, 100% agree with that. I also think testing it for some time on hardware before merging would be a good idea

[fedonman]

Great job to a quite difficult refactor!! 💯 💯

I have some ideas/questions to discuss with you as next steps, like if we could de-couple the compiler from the platform (don't pass the platform as a constructor parameter), but they are outside the scope of the PR.

Left some comments.

[fedonman]

💯 💯