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

QFT kata: fix formula formatting in square wave prep exercise #1819

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

tcNickolas
Copy link
Member

No description provided.

@tcNickolas
Copy link
Member Author

Current version in playground:

image

Fixed version:

image

Copy link

github-actions bot commented Aug 6, 2024

Benchmark for d215520

Click to view benchmark
Test Base PR %
Array append evaluation 335.8±4.97µs 338.7±2.84µs +0.86%
Array literal evaluation 186.3±2.44µs 169.0±2.88µs -9.29%
Array update evaluation 411.5±2.04µs 416.3±3.18µs +1.17%
Core + Standard library compilation 22.4±0.56ms 23.9±1.09ms +6.70%
Deutsch-Jozsa evaluation 5.0±0.07ms 5.0±0.06ms 0.00%
Large file parity evaluation 35.0±0.27ms 35.2±0.58ms +0.57%
Large input file compilation 13.7±0.50ms 13.6±0.68ms -0.73%
Large input file compilation (interpreter) 55.0±2.29ms 55.7±2.23ms +1.27%
Large nested iteration 33.1±0.30ms 33.1±0.23ms 0.00%
Perform Runtime Capabilities Analysis (RCA) on Deutsch-Jozsa sample 1586.7±64.89µs 1601.6±107.28µs +0.94%
Perform Runtime Capabilities Analysis (RCA) on large file sample 7.9±0.08ms 7.9±0.09ms 0.00%
Perform Runtime Capabilities Analysis (RCA) on teleport sample 1453.6±73.49µs 1447.9±59.46µs -0.39%
Perform Runtime Capabilities Analysis (RCA) on the core and std libraries 28.0±0.27ms 28.5±0.62ms +1.79%
Teleport evaluation 93.5±3.38µs 93.3±3.64µs -0.21%

@tcNickolas tcNickolas added this pull request to the merge queue Aug 6, 2024
Merged via the queue into main with commit 713401e Aug 6, 2024
19 checks passed
@tcNickolas tcNickolas deleted the mariia/fix-squarewave-formatting branch August 6, 2024 20:36
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.

3 participants