Skip to content

chore: photometry dedupe, .dockerignore, pytest in CI, real-code science tests (ARCH-01, DEPLOY-04, CI-01, TEST-01) - #9

Open
KARLalpha4768 wants to merge 1 commit into
mainfrom
chore/build-test-hygiene
Open

KARLalpha4768 wants to merge 1 commit into
mainfrom
chore/build-test-hygiene

Conversation

@KARLalpha4768

Copy link
Copy Markdown
Owner

Changes

  • ARCH-01: src/cpp/photometry.cpp stays canonical; src/photometry.cpp becomes a thin include so the legacy Docker build still compiles.
  • DEPLOY-04: root .dockerignore for git metadata, env files, venvs, node_modules, caches, tests, DBs, secrets/keys, CMake output.
  • CI-01: CI installs pytest, runs a repo-wide fatal-error ruff lint (excluding the known-invalid ascom_hal.py, fixed in Fix HIGH correctness bugs CORR-01..05 #5), runs pytest -q, and fails unknown CMake errors (only missing-CUDA is tolerated).
  • TEST-01: test_science_modules.py uses production OrbitalElements / OrbitPropagator.propagate instead of local toy math.

Tests

  • tests/test_build_hygiene.py.

Not verified

  • No C++ compiler was run for the forwarding include; CI workflow not yet run on GitHub (this PR's own run is the first check). The CI requirements.txt install must provide test deps.

Verification

Drafted by Codex from the medium/low review findings; split and re-verified per branch.

… test production orbit code (ARCH-01, DEPLOY-04, CI-01, TEST-01)
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.

1 participant