Revert unneeded changes

Author: pamelafox

.github/workflows/python-test.yaml

@@ -0,0 +1,74 @@
+name: Python check
+
+on:
+  push:
+    branches: [ main ]
+    paths-ignore:
+      - "**.md"
+      - ".azdo/**"
+      - ".devcontainer/**"
+      - ".github/**"
+  pull_request:
+    branches: [ main ]
+    paths-ignore:
+      - "**.md"
+      - ".azdo/**"
+      - ".devcontainer/**"
+      - ".github/**"
+  workflow_call:
+
+jobs:
+  test_package:
+    name: Test ${{ matrix.os }} Python ${{ matrix.python_version }}
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: ["ubuntu-20.04", "windows-latest"]
+        python_version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
+    steps:
+        - uses: actions/checkout@v4
+        - name: Install uv
+          uses: astral-sh/setup-uv@v5
+          with:
+            enable-cache: true
+            version: "0.4.20"
+            cache-dependency-glob: "requirements**.txt"
+            python-version: ${{ matrix.python_version }}
+        - name: Setup node
+          uses: actions/setup-node@v4
+          with:
+            node-version: 18
+        - name: Build frontend
+          run: |
+            cd ./app/frontend
+            npm install
+            npm run build
+        - name: Install dependencies
+          run: |
+            uv pip install -r requirements-dev.txt
+        - name: Lint with ruff
+          run: ruff check .
+        - name: Check types with mypy
+          run: |
+            cd scripts/
+            mypy . --config-file=../pyproject.toml
+            cd ../app/backend/
+            mypy . --config-file=../../pyproject.toml
+        - name: Check formatting with black
+          run: black . --check --verbose
+        - name: Run Python tests
+          if: runner.os != 'Windows'
+          run: pytest -s -vv --cov --cov-fail-under=86
+        - name: Run E2E tests with Playwright
+          id: e2e
+          if: runner.os != 'Windows'
+          run: |
+            playwright install chromium --with-deps
+            pytest tests/e2e.py --tracing=retain-on-failure
+        - name: Upload test artifacts
+          if: ${{ failure() && steps.e2e.conclusion == 'failure' }}
+          uses: actions/upload-artifact@v4
+          with:
+            name: playwright-traces${{ matrix.python_version }}
+            path: test-results

app/backend/approaches/approach.py

@@ -152,18 +152,7 @@ async def search(
     ) -> List[Document]:
         search_text = query_text if use_text_search else ""
         search_vectors = vectors if use_vector_search else []
-        import logging
-
-        logger = logging.getLogger("scripts")
         if use_semantic_ranker:
-            logger.info("Using semantic ranker")
-            logger.info("Search text: %s", search_text)
-            logger.info("Filter: %s", filter)
-            logger.info("Top: %s", top)
-            logger.info("Query language: %s", self.query_language)
-            logger.info("Query speller: %s", self.query_speller)
-            logger.info("Semantic configuration: %s", "default")
-            logger.info("Semantic query: %s", query_text)
             results = await self.search_client.search(
                 search_text=search_text,
                 filter=filter,
@@ -177,7 +166,6 @@ async def search(
                 semantic_query=query_text,
             )
         else:
-            logger.info("Not using semantic ranker")
             results = await self.search_client.search(
                 search_text=search_text,
                 filter=filter,