Skip to content

Add onboarding instructions for Copilot coding agents#50

Merged
GernotMaier merged 6 commits intomainfrom
copilot/add-copilot-instructions-file
Feb 17, 2026
Merged

Add onboarding instructions for Copilot coding agents#50
GernotMaier merged 6 commits intomainfrom
copilot/add-copilot-instructions-file

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 17, 2026

Adds .github/copilot-instructions.md to provide coding agents with essential context for effective contribution. Addresses the repository's domain complexity (gamma-ray astronomy + ML) and critical implementation details (dual telescope indexing modes).

Structure

  • Domain-specific sections for three reviewer personas:

    • Gamma-ray astronomy: air shower physics, VERITAS/CTAO telescope arrays, stereoscopic reconstruction
    • Machine learning: XGBoost regression/classification, energy-binned training, telescope multiplicity handling
    • Python/computing: ROOT→awkward→pandas pipeline, parallel decompression, variable-length array handling
  • Critical gotchas: Telescope indexing (VERITAS fixed vs CTAO variable), DispTelList_T mapping, NaN handling, feature sorting requirements

  • Verified workflows: Build/test/lint commands, CLI entry points, changelog management (towncrier), pre-commit hooks

Key Technical Details Documented

Telescope indexing (most complex logic):

  • VERITAS mode: Fixed indices with R_core branch
  • CTAO mode: Variable-length with ImgSel_list branch
  • _resolve_branch_aliases() and _normalize_telescope_variable_to_tel_id_space() handle compatibility

Data pipeline:

# Variable-length telescope data → dense arrays
flatten_feature_data()  # Sorts by mirror area, then size
# Missing telescopes padded with np.nan (XGBoost native support)

Test coverage: 27% overall (60% for core data_processing.py, 0% for config.py/models.py/evaluate.py)

Files Changed

  • .github/copilot-instructions.md: 492 lines covering project overview, conventions, domain knowledge, common patterns
  • .gitignore: Added .coverage test artifact

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 3 commits February 17, 2026 10:44
Co-authored-by: GernotMaier <14001688+GernotMaier@users.noreply.github.com>
Co-authored-by: GernotMaier <14001688+GernotMaier@users.noreply.github.com>
Co-authored-by: GernotMaier <14001688+GernotMaier@users.noreply.github.com>
Copilot AI changed the title [WIP] Add Copilot instructions for efficient onboarding Add onboarding instructions for Copilot coding agents Feb 17, 2026
Copilot AI requested a review from GernotMaier February 17, 2026 10:49
@GernotMaier GernotMaier marked this pull request as ready for review February 17, 2026 10:51
@GernotMaier GernotMaier merged commit a2f1bac into main Feb 17, 2026
2 checks passed
@GernotMaier GernotMaier deleted the copilot/add-copilot-instructions-file branch February 17, 2026 10:53
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