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

97 reorg files #98

Merged
merged 15 commits into from
Nov 28, 2024
Merged

97 reorg files #98

merged 15 commits into from
Nov 28, 2024

Conversation

jordandsullivan
Copy link
Collaborator

Fix some inconsistencies and formatting issues with data storage

Copy link
Member

@natestemen natestemen left a comment

Choose a reason for hiding this comment

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

Looking good. You likely also want to remove the json files in benchmarks/results.

benchmarks/run_ucc_benchmarks.ipynb Outdated Show resolved Hide resolved
benchmarks/scripts/common.py Outdated Show resolved Hide resolved
benchmarks/scripts/common.py Outdated Show resolved Hide resolved
benchmarks/scripts/pytket_compile.py Outdated Show resolved Hide resolved
benchmarks/scripts/gate_counters.py Outdated Show resolved Hide resolved
@jordandsullivan jordandsullivan merged commit 5b00437 into main Nov 28, 2024
@jordandsullivan jordandsullivan deleted the 97-reorg-files branch November 28, 2024 19:52
@jordandsullivan jordandsullivan linked an issue Dec 3, 2024 that may be closed by this pull request
jordandsullivan added a commit that referenced this pull request Dec 3, 2024
* Move and expand test for logical equivalence

Test moved to test_compile.py

* Delete test_circuit_equivalence.py

Test moved to test_compile.py

* Remove unnecessary qubit parameterizations

Remove unnecessary qubit parameterizations in `test_compiled_circuits_equivalent` and `test_compilation_retains_gateset`

* add basic expectation value benchmark (#96)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Add bug label to new bug report issues

* Add feature tag to feature template issues

* 97 reorg files (#98)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Update naming convention for different benchmarks

* Dry up save results into common function

* Fix init import issue

* Move gate_counters into scripts/common.py for simplicity

* Fix notebook errors, remove old data

* Remove duplicate funciton definitions

* Remove duplicate lines

* Add folder to docstring, call out alterntive benchmark name

---------

Co-authored-by: nate stemen <[email protected]>

---------

Co-authored-by: Misty-W <[email protected]>
Co-authored-by: nate stemen <[email protected]>
jordandsullivan added a commit that referenced this pull request Dec 3, 2024
* Move and expand test for logical equivalence

Test moved to test_compile.py

* Delete test_circuit_equivalence.py

Test moved to test_compile.py

* Remove unnecessary qubit parameterizations

Remove unnecessary qubit parameterizations in `test_compiled_circuits_equivalent` and `test_compilation_retains_gateset`

* add basic expectation value benchmark (#96)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Add bug label to new bug report issues

* Add feature tag to feature template issues

* 97 reorg files (#98)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Update naming convention for different benchmarks

* Dry up save results into common function

* Fix init import issue

* Move gate_counters into scripts/common.py for simplicity

* Fix notebook errors, remove old data

* Remove duplicate funciton definitions

* Remove duplicate lines

* Add folder to docstring, call out alterntive benchmark name

---------

Co-authored-by: nate stemen <[email protected]>

---------

Co-authored-by: Misty-W <[email protected]>
Co-authored-by: nate stemen <[email protected]>
jordandsullivan added a commit that referenced this pull request Dec 3, 2024
* Move and expand test for logical equivalence

Test moved to test_compile.py

* Delete test_circuit_equivalence.py

Test moved to test_compile.py

* Remove unnecessary qubit parameterizations

Remove unnecessary qubit parameterizations in `test_compiled_circuits_equivalent` and `test_compilation_retains_gateset`

* add basic expectation value benchmark (#96)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Add bug label to new bug report issues

* Add feature tag to feature template issues

* 97 reorg files (#98)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Update naming convention for different benchmarks

* Dry up save results into common function

* Fix init import issue

* Move gate_counters into scripts/common.py for simplicity

* Fix notebook errors, remove old data

* Remove duplicate funciton definitions

* Remove duplicate lines

* Add folder to docstring, call out alterntive benchmark name

---------

Co-authored-by: nate stemen <[email protected]>

---------

Co-authored-by: Misty-W <[email protected]>
Co-authored-by: nate stemen <[email protected]>
jordandsullivan added a commit that referenced this pull request Dec 3, 2024
* Initial dockerfile, breaking out requirements.

* Install requirements in Dockerfile, add -e pip flag

* Automated deploy script to AWS

* merge in new shell script changes (#104)

* Move and expand test for logical equivalence

Test moved to test_compile.py

* Delete test_circuit_equivalence.py

Test moved to test_compile.py

* Remove unnecessary qubit parameterizations

Remove unnecessary qubit parameterizations in `test_compiled_circuits_equivalent` and `test_compilation_retains_gateset`

* add basic expectation value benchmark (#96)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Add bug label to new bug report issues

* Add feature tag to feature template issues

* 97 reorg files (#98)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Update naming convention for different benchmarks

* Dry up save results into common function

* Fix init import issue

* Move gate_counters into scripts/common.py for simplicity

* Fix notebook errors, remove old data

* Remove duplicate funciton definitions

* Remove duplicate lines

* Add folder to docstring, call out alterntive benchmark name

---------

Co-authored-by: nate stemen <[email protected]>

---------

Co-authored-by: Misty-W <[email protected]>
Co-authored-by: nate stemen <[email protected]>

* Update PyTKET rebase pass

---------

Co-authored-by: Misty-W <[email protected]>
Co-authored-by: nate stemen <[email protected]>
jordandsullivan added a commit that referenced this pull request Dec 3, 2024
* remove unnecessary script

* merge in recent changes from main  (#106)

* Move and expand test for logical equivalence

Test moved to test_compile.py

* Delete test_circuit_equivalence.py

Test moved to test_compile.py

* Remove unnecessary qubit parameterizations

Remove unnecessary qubit parameterizations in `test_compiled_circuits_equivalent` and `test_compilation_retains_gateset`

* add basic expectation value benchmark (#96)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Add bug label to new bug report issues

* Add feature tag to feature template issues

* 97 reorg files (#98)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Update naming convention for different benchmarks

* Dry up save results into common function

* Fix init import issue

* Move gate_counters into scripts/common.py for simplicity

* Fix notebook errors, remove old data

* Remove duplicate funciton definitions

* Remove duplicate lines

* Add folder to docstring, call out alterntive benchmark name

---------

Co-authored-by: nate stemen <[email protected]>

---------

Co-authored-by: Misty-W <[email protected]>
Co-authored-by: nate stemen <[email protected]>

* merge in main (#107)

* Move and expand test for logical equivalence

Test moved to test_compile.py

* Delete test_circuit_equivalence.py

Test moved to test_compile.py

* Remove unnecessary qubit parameterizations

Remove unnecessary qubit parameterizations in `test_compiled_circuits_equivalent` and `test_compilation_retains_gateset`

* add basic expectation value benchmark (#96)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Add bug label to new bug report issues

* Add feature tag to feature template issues

* 97 reorg files (#98)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Update naming convention for different benchmarks

* Dry up save results into common function

* Fix init import issue

* Move gate_counters into scripts/common.py for simplicity

* Fix notebook errors, remove old data

* Remove duplicate funciton definitions

* Remove duplicate lines

* Add folder to docstring, call out alterntive benchmark name

---------

Co-authored-by: nate stemen <[email protected]>

---------

Co-authored-by: Misty-W <[email protected]>
Co-authored-by: nate stemen <[email protected]>

* Delete benchmarks/scripts/deploy_benchmark_task.py

No longer needed, build-spec lives on AWS

* remove unnecessary .gitignore line

---------

Co-authored-by: Misty-W <[email protected]>
Co-authored-by: nate stemen <[email protected]>
jordandsullivan added a commit that referenced this pull request Dec 6, 2024
* Add a couple comments, correct typo

* Add final basis set remapping step to transpilation when passed a Qiskit backend

* First hardware benchmark notebook

* merge in main (#109)

* Initial dockerfile, breaking out requirements.

* Install requirements in Dockerfile, add -e pip flag

* Automated deploy script to AWS

* merge in new shell script changes (#104)

* Move and expand test for logical equivalence

Test moved to test_compile.py

* Delete test_circuit_equivalence.py

Test moved to test_compile.py

* Remove unnecessary qubit parameterizations

Remove unnecessary qubit parameterizations in `test_compiled_circuits_equivalent` and `test_compilation_retains_gateset`

* add basic expectation value benchmark (#96)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Add bug label to new bug report issues

* Add feature tag to feature template issues

* 97 reorg files (#98)

* add basic expectation value benchmark

* move compile function to `common.py`

* write data out to json

* make data pipeline more functional

* simplify real check

* Update naming convention for different benchmarks

* Dry up save results into common function

* Fix init import issue

* Move gate_counters into scripts/common.py for simplicity

* Fix notebook errors, remove old data

* Remove duplicate funciton definitions

* Remove duplicate lines

* Add folder to docstring, call out alterntive benchmark name

---------

Co-authored-by: nate stemen <[email protected]>

---------

Co-authored-by: Misty-W <[email protected]>
Co-authored-by: nate stemen <[email protected]>

* Update PyTKET rebase pass

---------

Co-authored-by: Misty-W <[email protected]>
Co-authored-by: nate stemen <[email protected]>

* Real hardware benchmark

* Remove unrelated file

* Remove unnecessary .gitignore line

---------

Co-authored-by: Misty-W <[email protected]>
Co-authored-by: nate stemen <[email protected]>
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.

Reorganize results files
2 participants