Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added filters on the runcard #768

Conversation

jordivallsq
Copy link
Contributor

Created filters in the controlers section of QUA config

@jordivallsq jordivallsq self-assigned this Aug 2, 2024
Copy link

linear bot commented Aug 2, 2024

Copy link

codecov bot commented Aug 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.02%. Comparing base (e8b73c6) to head (9252d11).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #768   +/-   ##
=======================================
  Coverage   95.02%   95.02%           
=======================================
  Files         266      266           
  Lines        8724     8724           
=======================================
  Hits         8290     8290           
  Misses        434      434           
Flag Coverage Δ
unittests 95.02% <ø> (ø)

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.

Copy link
Contributor

@GuillermoAbadLopez GuillermoAbadLopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look really nice! GJ

Left 2 silly comments!

docs/releases/changelog-dev.md Show resolved Hide resolved
docs/releases/changelog-dev.md Outdated Show resolved Hide resolved
jordivallsq and others added 3 commits August 7, 2024 14:16
Co-authored-by: Guillermo Abad López <109400222+GuillermoAbadLopez@users.noreply.github.com>
@jordivallsq
Copy link
Contributor Author

Adressed all the comments

Copy link
Contributor

@GuillermoAbadLopez GuillermoAbadLopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GJ!

Just one last thing to adress before testing on hardware and merge! 🙌

docs/releases/changelog-dev.md Outdated Show resolved Hide resolved
@jordivallsq
Copy link
Contributor Author

The filters have been tested in QM OPX and the results look the same as the ones measured by Mireia.

Before filters:
imagen

After filters:
imagen

@jordivallsq jordivallsq merged commit d649a8b into main Aug 8, 2024
6 checks passed
@jordivallsq jordivallsq deleted the qhc-634-distortion-coefficients-in-runcard-gets-mapped-to-qm-driver branch August 8, 2024 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants