Fix CI Python matrix failures from stray syntax in space_server.py

[Copilot]

The CI/CD Pipeline was failing across Python 3.10–3.13 before tests ran because Ruff hit an invalid-syntax error in space_server.py. The failure came from a duplicated, malformed fragment left between the VQE and noisy simulation endpoints.

• Root cause

  • Removed the stray block that introduced unexpected indentation and broke parsing in the VQE/noisy endpoint boundary.

• VQE endpoint cleanup

  • Reduced the VQE fallback handler to a single coherent implementation.
  • Removed duplicated dead code left from the broken fragment.
  • Moved the NumPy import to module scope and aligned the function body with the repository’s Ruff formatting rules.

• Behavioral scope

  • Keeps the existing simplified VQE response shape intact (exact_energy, vqe_energy, error, depth, iterations, mode).
  • Limits the change to space_server.py; no workflow or test code changes.

# Before: duplicated malformed fragment after the VQE handler
except Exception as e:
    raise HTTPException(status_code=500, detail=str(e)) from e

# Noisy Simulation Endpoint
            }
    except Exception as e:
        raise HTTPException(status_code=500, detail=str(e)) from e

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
q-pyth	Ready	Preview, Comment	Apr 5, 2026 4:23am

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[Copilot]

Pull request overview

Fixes CI failures across the Python matrix by removing an invalid/stray code fragment in space_server.py and cleaning up the VQE endpoint implementation.

Changes:

• Removed malformed duplicated fragment between the VQE and noisy simulation endpoints that caused invalid-syntax.
• Simplified the VQE fallback handler by removing duplicated/dead code and aligning formatting.
• Moved numpy import to module scope and normalized math/return formatting for Ruff.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

shots is accepted (and passed into simplified_vqe) but no longer used after removing the AerSimulator execution. This makes the API parameter effectively a no-op; either incorporate shots into the simplified simulation (e.g., scale the noise/error with shots) or remove/deprecate the parameter so clients aren’t misled.

[Copilot]

The H₂ ground-state energy approximation is duplicated (computed as exact inside simplified_vqe and again as exact_energy outside). Consider computing it once (or extracting a small helper) to avoid the two formulas drifting if updated in the future.