Repository: zapata-engineering/orquestra-quantum (language: python)

Issue:
fix: remove unnecessary print
## Description
Removed unnecessary print.

## Please verify that you have completed the following steps

- [X] I have self-reviewed my code.



Discussion / hints:

Fix patch:
diff --git a/setup.cfg b/setup.cfg
index f1e4c29..3cd0bf9 100644
--- a/setup.cfg
+++ b/setup.cfg
@@ -38,7 +38,7 @@ where = src
 # TODO: this is a temporary solution until we find a good way 
 # to have consistent dev requirements across multiple repositories
 [options.extras_require]
-develop =
+dev =
     black~=22.3
     flake8~=3.9.0
     isort~=5.9.0
diff --git a/subtrees/z_quantum_actions/Makefile b/subtrees/z_quantum_actions/Makefile
index 0f606cd..03c1f20 100644
--- a/subtrees/z_quantum_actions/Makefile
+++ b/subtrees/z_quantum_actions/Makefile
@@ -1,3 +1,6 @@
+################################################################################
+# © Copyright 2021-2022 Zapata Computing Inc.
+################################################################################
 TOP_DIR := $(realpath $(dir $(lastword $(MAKEFILE_LIST))))
 include $(TOP_DIR)/variables.mk
 
@@ -42,14 +45,13 @@ install-default: clean
 	$(PYTHON) -m pip install --upgrade pip && \
 		$(PYTHON) -m pip install .
 
-# Renamed to develop to distinguish from dev branch
-develop-default: clean
-	$(PYTHON) -m pip install -e .[develop]
+dev-default: clean
+	$(PYTHON) -m pip install -e .[dev]
 
 github_actions-default:
 	python3 -m venv ${VENV} && \
 		${VENV}/bin/python3 -m pip install --upgrade pip && \
-		${VENV}/bin/python3 -m pip install -e '.[develop]'
+		${VENV}/bin/python3 -m pip install -e '.[dev]'
 
 flake8-default: clean
 	$(PYTHON) -m flake8 --ignore=E203,E266,F401,W503 --max-line-length=88 src tests
@@ -82,6 +84,9 @@ style-default: flake8 mypy black isort
 muster-default: style coverage
 	@echo This project passes muster!
 
+build-system-deps-default:
+	:
+
 # This is what converts the -default targets into base target names.
 # Do not remove!!!
 %: %-default
diff --git a/subtrees/z_quantum_actions/README.rst b/subtrees/z_quantum_actions/README.rst
index 53f21da..413e228 100644
--- a/subtrees/z_quantum_actions/README.rst
+++ b/subtrees/z_quantum_actions/README.rst
@@ -20,7 +20,7 @@ Files for Python development:
    * pytest.ini: A sample pytest.ini, can be symlinked
    * MANIFEST.in: Standard set of non-Python files that should be included in distributions. To be symlinked in top level
    * sample_setup.py: a sample setup.py file
-   * setup_extras.py: extra imports for setup.py develop mode
+   * setup_extras.py: extra imports for setup.py dev mode
    * variables.mk: Makefile global variables
 
 
@@ -128,9 +128,6 @@ include, just add your override. For example ::
 Configuring setup.py
 --------------------------
 
-.. Note:: We have labeled the development target as *develop* to avoid confusion
-   with the *dev* branch name.
-
 * Backup your original setup.py
 * Copy current sample_setup.py to your repo's setup.py in your target repo.
 
@@ -140,14 +137,14 @@ Inside of setup.py we need to make sure the extras are in place::
         from subtrees.z_quantum_actions.setup_extras import extras
     except ImportError:
         print("Unable to import extras", file=sys.stderr)
-        extras = {"develop": []}
+        extras = {"dev": []}
 
 If you have other additions to extras you need to extend the standardized ones
 provided by this repo. Inside your *setup()* do ::
 
     extras_require={
-        "develop": [
-            *extras["develop"],
+        "dev": [
+            *extras["dev"],
             "my_requirement~=1.2",
         ],
     }
diff --git a/subtrees/z_quantum_actions/actions/publish-release/action.yml b/subtrees/z_quantum_actions/actions/publish-release/action.yml
index 1d88650..11877b1 100644
--- a/subtrees/z_quantum_actions/actions/publish-release/action.yml
+++ b/subtrees/z_quantum_actions/actions/publish-release/action.yml
@@ -34,12 +34,46 @@ runs:
         git push --tags
         echo "::set-output name=tag::$TAG"
 
+    # Use pip to build a PEP 517 wheel
+    - name: Build wheel
+      id: build-wheel
+      shell: bash
+      run: python3 -m pip wheel --isolated --no-deps -w dist .
+
+    # Upload all artifacts in dist
+    - name: Store wheel artifacts
+      uses: actions/upload-artifact@v2
+      with:
+        name: Python Wheels
+        path: dist
+
     - name: Create entry on GH Releases
+      id: make-release
       shell: bash
       run: |
-        curl \
+        # When we make a release, we should check the response for the "upload_url"
+        API_RESPONSE=$(curl \
           -X POST \
-          curl -H "Authorization: token ${{ github.token }}" \
+          -H "Authorization: token ${{ github.token }}" \
           -H "Accept: application/vnd.github.v3+json" \
           https://api.github.com/repos/${{ github.repository }}/releases \
-          -d '{"tag_name":"${{ steps.push-new-version-tag.outputs.tag }}"}'
+          -d '{"tag_name":"${{ steps.push-new-version-tag.outputs.tag }}"}')
+        # The URL from github has some extras in curly brackets that we do not want:
+        # The URL always ends in assets, so we can use that to help check for the end
+        UPLOAD_URL=$(echo ${API_RESPONSE} | jq .upload_url | sed "s/\(.*assets\){.*}/\1/")
+        echo "::set-output name=upload_url::${UPLOAD_URL}"
+
+    - name: Upload wheel to release page
+      shell: bash
+      run: |
+        # We don't know the wheel name, so we can check for everything in the dist dir
+        # We may build multiple wheels from a single repo, so this works for that too.
+        for wheel in dist/*; do
+          curl \
+            -X POST \
+            -H "Authorization: token ${{ github.token }}" \
+            -H "Accept: application/vnd.github.v3+json" \
+            -H "Content-Type: $(file -b --mime-type $wheel)" \
+            -T "$wheel" \
+            "${{ steps.make-release.outputs.upload_url }}?name=$(basename $wheel)"
+        done
diff --git a/subtrees/z_quantum_actions/bin/get_next_version.py b/subtrees/z_quantum_actions/bin/get_next_version.py
index c020a20..c646611 100755
--- a/subtrees/z_quantum_actions/bin/get_next_version.py
+++ b/subtrees/z_quantum_actions/bin/get_next_version.py
@@ -1,4 +1,7 @@
 #! /usr/bin/env python3
+################################################################################
+# © Copyright 2022 Zapata Computing Inc.
+################################################################################
 
 # This script ia intended to be run from within a Github Action.
 # Reads current project version, bumps "minor", and sets the "next_version" output
diff --git a/subtrees/z_quantum_actions/sample_setup.py b/subtrees/z_quantum_actions/sample_setup.py
index 80f8e3c..a6b4b35 100644
--- a/subtrees/z_quantum_actions/sample_setup.py
+++ b/subtrees/z_quantum_actions/sample_setup.py
@@ -1,4 +1,7 @@
 #!/usr/bin/env python
+################################################################################
+# © Copyright 2021-2022 Zapata Computing Inc.
+################################################################################
 
 import site
 import sys
@@ -10,7 +13,7 @@
     from subtrees.z_quantum_actions.setup_extras import extras
 except ImportError:
     print("Unable to import extras", file=sys.stderr)
-    extras = {"develop": []}
+    extras = {"dev": []}
 
 # Workaound for https://github.com/pypa/pip/issues/7953
 site.ENABLE_USER_SITE = "--user" in sys.argv[1:]
diff --git a/subtrees/z_quantum_actions/setup_extras.py b/subtrees/z_quantum_actions/setup_extras.py
index ed1d1da..8013ba7 100644
--- a/subtrees/z_quantum_actions/setup_extras.py
+++ b/subtrees/z_quantum_actions/setup_extras.py
@@ -1,8 +1,10 @@
+################################################################################
+# © Copyright 2021-2022 Zapata Computing Inc.
+################################################################################
 extras = {
     # Development extras needed in every project, because the stylechecks depend on it.
     # If you need more dev deps, extend this list in your own setup.py.
-    # Using "develop" instead of "dev" to distinguish from branch name.
-    "develop": [
+    "dev": [
         "black~=22.3",
         "flake8~=3.9.0",
         "isort~=5.9.0",


Test patch:
diff --git a/src/orquestra/quantum/testing/__init__.py b/src/orquestra/quantum/testing/__init__.py
index 2206324..abc2b78 100644
--- a/src/orquestra/quantum/testing/__init__.py
+++ b/src/orquestra/quantum/testing/__init__.py
@@ -1,1 +1,2 @@
 from ._initialize import *  # noqa: F403
+from .mocks import MockQuantumBackend
diff --git a/src/orquestra/quantum/testing/mocks.py b/src/orquestra/quantum/testing/mocks.py
index 09e0ea2..d035da7 100644
--- a/src/orquestra/quantum/testing/mocks.py
+++ b/src/orquestra/quantum/testing/mocks.py
@@ -4,10 +4,6 @@
 from orquestra.quantum.symbolic_simulator import SymbolicSimulator
 
 
-def mock_create_noise_model(noise_model_data):
-    return None
-
-
 class MockQuantumBackend(QuantumBackend):
 
     supports_batching = False
diff --git a/subtrees/z_quantum_actions/tests/test_get_next_version.py b/subtrees/z_quantum_actions/tests/test_get_next_version.py
index 9fb7acd..c2f3fa4 100644
--- a/subtrees/z_quantum_actions/tests/test_get_next_version.py
+++ b/subtrees/z_quantum_actions/tests/test_get_next_version.py
@@ -1,3 +1,6 @@
+################################################################################
+# © Copyright 2022 Zapata Computing Inc.
+################################################################################
 import re
 import subprocess
 
diff --git a/tests/orquestra/quantum/estimation/estimation_test.py b/tests/orquestra/quantum/estimation/estimation_test.py
index 146eed4..c1ad8f7 100644
--- a/tests/orquestra/quantum/estimation/estimation_test.py
+++ b/tests/orquestra/quantum/estimation/estimation_test.py
@@ -17,9 +17,9 @@
     split_estimation_tasks_to_measure,
 )
 from orquestra.quantum.measurements import ExpectationValues, Measurements
-from orquestra.quantum.mock_objects import MockQuantumBackend
 from orquestra.quantum.openfermion import IsingOperator, QubitOperator
 from orquestra.quantum.symbolic_simulator import SymbolicSimulator
+from orquestra.quantum.testing import MockQuantumBackend
 
 
 class TestEstimatorUtils: