Skip to content

Document mask and clustering feature extraction - #1324

Draft
alecandido wants to merge 14 commits into
mainfrom
document-mask-clustering
Draft

Document mask and clustering feature extraction#1324
alecandido wants to merge 14 commits into
mainfrom
document-mask-clustering

Conversation

@alecandido

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented Dec 22, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.86%. Comparing base (6988e24) to head (143f875).

Additional details and impacted files

Impacted file tree graph

@@                      Coverage Diff                      @@
##           stabilize-extract-feature    #1324      +/-   ##
=============================================================
+ Coverage                      97.84%   97.86%   +0.01%     
=============================================================
  Files                            132      132              
  Lines                          10136    10135       -1     
=============================================================
+ Hits                            9918     9919       +1     
+ Misses                           218      216       -2     
Flag Coverage Δ
unittests 97.86% <100.00%> (+0.01%) ⬆️

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

Files with missing lines Coverage Δ
src/qibocal/protocols/utils.py 93.05% <100.00%> (-0.02%) ⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lballerio lballerio assigned alecandido and unassigned alecandido Jan 2, 2026
@lballerio

Copy link
Copy Markdown
Contributor

@alecandido in the new commit I tested the signal detection for ResonatorPunchout without using the mask...as you can see we see the signal in the two distinct regimes (high and low amplitudes), but still we cannot find the intermediate regime where the interaction is slowing switching on.
Why don't we save indeed the two clusters separately?

@alecandido

Copy link
Copy Markdown
Member Author

@lballerio as mentioned, the case of the "punchout" (aka power shift) is quite different from the flux dependencies.

Indeed, we do not have (or at least apply) a global model for the behavior of the lower-dimensional feature. Even in the flux-dependencies, we do not have a model for the full 2-dimensional behavior. But once we extract the 1-dimensional feature, we can model it all.

However, for the power shift, our assumption is just that we have two different regimes, with two different resonance frequencies. And they should be steady, in a certain range.
I.e. from a certain point on, it does not matter how much power you give (until you break it), the resonator will be in the "high power" regime, with a fixed frequency. Same for the low-power, until your signal is killed by SNR.

So, our intended fit will consist of two vertical lines[*], once we have identified the two regimes. We are not modelling what is happening in the middle. And there is not much incentive to do that, since we will not consume this information: for us, this experiment is mostly useful to identify the bare and dressed frequency. Thus, the $\omega_r$ frequency of the resonator, and the dispersive shift $\chi$.

It's perfectly fine, and advised, to process the two clusters separately. Given that we are able to actually separate them in advance, automatically.

[*]: In that case, we may even have a 2D model, in a restricted area, since for each cut they should be fixed resonance shapes. Lorentzians, if you are lucky enough. But there are not many guarantees about the parameters which are not strictly related to the resonator itself, e.g. as the baseline.

@lballerio
lballerio force-pushed the document-mask-clustering branch from b7efbc6 to 796539b Compare January 6, 2026 17:02
@lballerio
lballerio force-pushed the document-mask-clustering branch from 796539b to 143f875 Compare January 7, 2026 10:54
Base automatically changed from stabilize-extract-feature to main January 12, 2026 14:23
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