-
-
Notifications
You must be signed in to change notification settings - Fork 32
✨ [NA] Adding evaluation script for zoned neutral atom compiler #875
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
Merged
Merged
Changes from 146 commits
Commits
Show all changes
147 commits
Select commit
Hold shift + click to select a range
c20a742
✨ Add max filling factor
ystade 1620370
✅ Add test for added behaviour
ystade ec0d8e8
Apply suggestions from code review
ystade c69f26e
🎨 pre-commit fixes
pre-commit-ci[bot] 0252833
🎨 Remove leftover
ystade c94ac78
🎨 Fix missing parking offset
ystade 5ee5b7c
🎨 Fix type conversion bug on windows
ystade 708d11d
✅ Increase test coverage
ystade 6dfb970
Merge branch 'main' into ystade/max-filling-factor
ystade a8c042c
📝 Add changelog entry
ystade 61688dd
✨ Enable relaxed rearrangements in code-generator
ystade 77e2fbc
✅ Adapt tests
ystade 9a05354
✅ Fix tests
ystade 03d2c8c
💚 Fix clang-tidy warnings
ystade 064eef4
🐛 Fix compilation bug on old mac OS
ystade aad203b
✅ Try increase test coverage
ystade 4936455
🐛 Fix bug
ystade 97c60a5
🐛 Fix missing constructor bug
ystade fc8a2d9
🐛 Fix logical bug
ystade 40c596f
✅ Add test for better coverage
ystade 068f659
Merge remote-tracking branch 'origin/main' into ystade/relaxed-code-g…
ystade 3e7c7bc
🎨 Some fixes/improvements
ystade 1353fc4
📝 Update changelog
ystade 4890a6d
🎨 Fix enumerate for rvalues
ystade a7e891c
🎨 Add Upgrading note
ystade 18ef464
🎨 Implement relaxed router
ystade 5fceb33
Merge remote-tracking branch 'origin/main' into ystade/relaxed-router
ystade 16ec923
🎨 Remove references from chrono variable
ystade 2d46808
📝 Add docstrings
ystade 1740419
📝 Add docstring
ystade 362bee6
🎨 Make function static
ystade 9575f66
🎨 More readable alternative
ystade c4e58a9
📝 Add addtional comment
ystade 00a4121
🔧 Add missing config option
ystade 1eac09f
🎨 Improve platform compatibility
ystade 85212ba
🎨 Add override annotations
ystade d19af9a
🎨 Fix minor bug
ystade 272c0d2
Merge branch 'ystade/relaxed-router' into ystade/ids-placer
ystade 77ebe3b
🚚 Rename AStarPlacer to HeuristicPlacer
ystade f5607f6
🚚 Update header to combine astar and ids placer
ystade cc9b214
🎨 Update HeuristicPlacer.cpp
ystade 013801f
📝 Refine docstring
ystade b80464f
📝 Correct docstring
ystade 4b440b8
📝 Fix comment
ystade e5db010
Merge remote-tracking branch 'origin/main' into ystade/relaxed-router
ystade 98d1bd0
🎨 Combine both routers
ystade a9f018d
🎨 Adapt test suite and fix bugs
ystade 309194a
🎨 Fix docstring
ystade 7e5feaf
🎨 Fix strict router
ystade ba19512
🎨 Fix json parsing
ystade f722105
🎨 Expose routing method
ystade b1379bd
🎨 Adopt python inteface file
ystade 3edebd7
💚 Add missing header
ystade e649e18
Merge branch 'ystade/relaxed-router' into ystade/ids-placer
ystade acdfa87
🎨 Add serialization for placement method
ystade 703f9c9
🎨 Expose placement method
ystade b70c807
🎨 Adopt python interface
ystade dfc8370
🎨 Adopting the tests and fixing minor bugs
ystade de7ef5e
📝 Fix docstrings
ystade b556d1b
🎨 Refactor code
ystade 2b53107
📝 Update changelog
ystade 17fbb32
📝 Make docstring more comprehensible
ystade 41eb559
📝 Fix changelog
ystade a4f4695
✅ Fix a star test
ystade 5f07b8d
🎨 Avoid overflow
ystade d0aba74
Merge branch 'ystade/relaxed-router' into ystade/ids-placer
ystade e80dcf0
🎨 Move templated function to header
ystade b37d570
🐛 Fix placement method
ystade 4684fb1
🎨 Split routing function
ystade 6f4b011
Merge branch 'ystade/relaxed-router' into ystade/ids-placer
ystade a39d267
🎨 Fix parsing
ystade f3e4399
⚡️ Micro optimization
ystade 342b89d
💚 Add missing headers
ystade a102c6b
📝 Make docstrings more precise
ystade b617658
📝 Mention preferSPlit parameter
ystade 837b393
⚡️ Improve performance
ystade d35f58d
📝 Provide defaults andd improve docstring
ystade d301fd3
📝 Improve docstring
ystade 3b7c22c
Merge branch 'main' into ystade/relaxed-router
ystade 039797b
🐛 Fix docstring and ordering of atoms to route
ystade 01fee79
Merge remote-tracking branch 'origin/main' into ystade/relaxed-router
ystade a9274c7
📝 Improve docstrings
ystade 1de1a75
Merge remote-tracking branch 'origin/main' into ystade/relaxed-router
ystade 8144152
Merge branch 'ystade/relaxed-router' into ystade/ids-placer
ystade 4b82679
🐛 Fix mac os 14 build
ystade a72dafe
🎨 Remove superfluous capture
ystade fb88af1
🎨 Micro opt
ystade 7765014
Merge branch 'ystade/relaxed-router' into ystade/ids-placer
ystade a3358ed
🎨 Apply suggestions from code review
ystade 24cd390
🎨 Apply suggestions from code review
ystade bd38d30
🎨 Add evaluation script
ystade aa42711
🚚 Rename evaluation script
ystade a9368b8
🚚 Rename evaluation script
ystade fd3d75e
🎨 Remove leftovers from parking_offset
ystade 8bb70ea
🎨 Expose trials and queueCapacity
ystade c6267a9
🎨 Expose prefer_split
ystade e0ef278
🎨 Clean-up eval script
ystade 37006fa
🎨 Remove leftovers from parking_offset
ystade 122fe45
🎨 Expose prefer_split
ystade 7389950
Merge branch 'ystade/relaxed-router' into ystade/ids-placer
ystade 9a447cb
🎨 Expose trials and queueCapacity
ystade 4f3b92c
Merge branch 'ystade/ids-placer' into ystade/eval-routing-aware
ystade 57e8dcf
🎨 Fix pyi file
ystade ff1bb53
🐛 Fix pyi file
ystade fd4cbf4
Merge branch 'ystade/relaxed-router' into ystade/ids-placer
ystade c85d7cc
Merge branch 'ystade/ids-placer' into ystade/eval-routing-aware
ystade aeb3d9f
➕ Add architecture file
ystade 22a58b2
🐛 Fix queue capacity parameter
ystade ab6e623
🐛 Fix queue capacity parameter
ystade be4788e
Merge branch 'ystade/ids-placer' into ystade/eval-routing-aware
ystade ef7216f
🎨 Update evaluation script
ystade 51b2acc
➕ Add naviz style
ystade 46fa195
🎨 Finalize evaluation script
ystade 36bb784
🎨 Apply coderabbit's suggestion
ystade 9722e5c
Merge remote-tracking branch 'origin/main' into ystade/eval-routing-a…
ystade a673ab7
📝 Improve docstring
ystade ac91302
🎨 Apply coderabbit's suggestion
ystade 0b02d53
📝 Improve docstring
ystade 99a30ed
Merge remote-tracking branch 'origin/main' into ystade/relaxed-router
ystade 747195e
Revert "🎨 Apply coderabbit's suggestion"
ystade 3d9a446
🎨 Apply coderabbit's suggestion
ystade 18dfdcf
📝 Improve docstring
ystade 7452910
👷 Update C++ test matrices (#874)
denialhaag b1d8928
📝 Update Changelog
ystade ae01d18
Revert "🎨 Apply coderabbit's suggestion"
ystade add9662
Merge branch 'ystade/relaxed-router' into ystade/ids-placer
ystade df48b85
Merge branch 'ystade/ids-placer' into ystade/eval-routing-aware
ystade 0db8c4e
🎨 Remove stray changes
ystade f04b572
🐛 Fix pyi
ystade c9546bd
🚚 Rename variable for clarity
ystade ff5b920
Merge branch 'ystade/relaxed-router' into ystade/ids-placer
ystade 2a74386
Merge branch 'ystade/ids-placer' into ystade/eval-routing-aware
ystade 542b452
Merge remote-tracking branch 'origin/main' into ystade/ids-placer
ystade 9c84575
Merge branch 'ystade/ids-placer' into ystade/eval-routing-aware
ystade 6019b97
📝 Improve docstring
ystade 17176b0
📝 Improve docstring
ystade bcb66f1
Merge branch 'ystade/ids-placer' into ystade/eval-routing-aware
ystade a0f8ca9
⬆️👨💻 Update actions/download-artifact action to v7 (#878)
renovate[bot] ab83069
✨ [NA] Add IDS Placement Method (#862)
ystade 2e72e10
⬆️🪝 Update patch updates (#877)
renovate[bot] 27f5c53
Merge remote-tracking branch 'origin/main' into ystade/eval-routing-a…
ystade 61a77ab
🎨 Suggestions from code review
ystade 52f6b42
⚡️ Add README, improve script
ystade 8c1651b
📝 Add comment
ystade bb44bf9
🎨 Improve docstring
ystade b052243
🎨 Apply suggestions from code review
ystade 7279666
Apply suggestions from code review
ystade File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| # MQT QMAP's Zoned Neutral Atom Compiler Evaluation | ||
|
|
||
| This document describes how to use the evaluation script for the zoned neutral atom compiler. | ||
| The script automates the process of running QMAP on a set of benchmark circuits and collecting performance metrics. | ||
|
|
||
| ## Prerequisites | ||
|
|
||
| Before running the evaluation script, ensure you have the following installed: | ||
|
|
||
| - uv 0.9.13 | ||
|
|
||
| ## Usage | ||
|
|
||
| The main evaluation script is `eval_ids_relaxed_routing.py`. You can run it from the repository root via: | ||
|
|
||
| ```bash | ||
| uv run eval/na/zoned/eval_ids_relaxed_routing.py | ||
ystade marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| ``` | ||
coderabbitai[bot] marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| ## Output | ||
|
|
||
| The script produces a CSV file named `results.csv` in the directory of the script. | ||
| Additionally, all input circuits are written to the `in` directory as QASM files. | ||
| Furthermore, the compiled files are written to the `out` directory ordered by compiler and settings. | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.