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

Dynamic loop insertion #614

Merged
merged 14 commits into from
Nov 22, 2021
Merged

Dynamic loop insertion #614

merged 14 commits into from
Nov 22, 2021

Conversation

terrorfisch
Copy link
Member

@terrorfisch terrorfisch commented Nov 15, 2021

  • New function qupulse._program._loop.roll_constant_waveforms that can transform long constant waveforms into repetitions of shorter waveforms.
  • The sample rate of HDAWG playbacks can now be dynamically reduced. Currently this can be enabled with the class attribute qupulse._program.seqc.WaveformPlayback.ENABLE_DYNAMIC_RATE_REDUCTION. Automatically use reduced sample for HDAWG #622

Tests pass

Todo:

  • newspiece for towncier

@coveralls
Copy link

coveralls commented Nov 15, 2021

Pull Request Test Coverage Report for Build 2426

  • 68 of 71 (95.77%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 80.808%

Changes Missing Coverage Covered Lines Changed/Added Lines %
qupulse/_program/volatile.py 3 4 75.0%
qupulse/_program/seqc.py 34 36 94.44%
Totals Coverage Status
Change from base Build 2420: 0.1%
Covered Lines: 7611
Relevant Lines: 9230

💛 - Coveralls

@terrorfisch terrorfisch marked this pull request as ready for review November 16, 2021 15:03
@terrorfisch
Copy link
Member Author

I have not yet found the intended way to link the github actions test result here, but there the tests pass.

https://github.com/qutech/qupulse/runs/4242052498

@terrorfisch terrorfisch merged commit 5e8c189 into master Nov 22, 2021
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