Skip to content

Add CI/CD pipeline and comprehensive test suite - #2

Open
rwtamCaltech wants to merge 1 commit into
masterfrom
claude/add-cicd-tests-QN0KQ
Open

Add CI/CD pipeline and comprehensive test suite#2
rwtamCaltech wants to merge 1 commit into
masterfrom
claude/add-cicd-tests-QN0KQ

Conversation

@rwtamCaltech

Copy link
Copy Markdown
Owner

Add 90 unit and integration tests covering:

  • detect_peaks: peak detection with various parameters (mph, mpd, threshold, valleys)
  • postprocess: extract_picks, calc_metrics, calc_timestamp
  • data_reader: normalize, normalize_long, normalize_batch, DataConfig
  • util: EMA, LMA, metrics, clean_queue
  • gamma._base: _check_shape, _check_X input validation
  • gamma.utils: convert_picks_csv, from_seconds, real data integrity checks

Add GitHub Actions CI/CD workflow with:

  • Lint job (Python syntax checking)
  • Unit tests across Python 3.9/3.10/3.11
  • Integration tests (runs after unit tests pass)
  • Full suite with coverage reporting
  • Pip caching and JUnit XML test result artifacts

Also add: requirements.txt, pytest.ini, .gitignore, test fixtures (conftest.py)

https://claude.ai/code/session_01P5P2JWzJQ3LcWG8Zhw44ob

Add 90 unit and integration tests covering:
- detect_peaks: peak detection with various parameters (mph, mpd, threshold, valleys)
- postprocess: extract_picks, calc_metrics, calc_timestamp
- data_reader: normalize, normalize_long, normalize_batch, DataConfig
- util: EMA, LMA, metrics, clean_queue
- gamma._base: _check_shape, _check_X input validation
- gamma.utils: convert_picks_csv, from_seconds, real data integrity checks

Add GitHub Actions CI/CD workflow with:
- Lint job (Python syntax checking)
- Unit tests across Python 3.9/3.10/3.11
- Integration tests (runs after unit tests pass)
- Full suite with coverage reporting
- Pip caching and JUnit XML test result artifacts

Also add: requirements.txt, pytest.ini, .gitignore, test fixtures (conftest.py)

https://claude.ai/code/session_01P5P2JWzJQ3LcWG8Zhw44ob
@rwtamCaltech rwtamCaltech self-assigned this Feb 11, 2026
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.

2 participants