Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 27, 2025

Description

Adds .github/copilot-instructions.md to provide repository-specific context for GitHub Copilot, as documented in Best practices for Copilot coding agent.

The instructions cover:

  • Project overview (MMM, CLV, Customer Choice, Bass models)
  • Project structure and key directories
  • Development setup (conda, make init, pre-commit)
  • Code style (Ruff, mypy, NumPy docstrings with both doctest and Sphinx code block examples, type hints)
  • Testing and documentation commands (with environment activation guidance)
  • Common patterns for model building and priors (using pymc_extras.prior)
  • Key dependencies (referencing pyproject.toml for versions)
  • PR guidelines and resource links

Related Issue

Checklist

Original prompt

This section details on the original issue you should resolve

<issue_title>✨ Set up Copilot instructions</issue_title>
<issue_description>Configure instructions for this repository as documented in Best practices for Copilot coding agent in your repository.

</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: williambdean <57733339+williambdean@users.noreply.github.com>
Copilot AI changed the title [WIP] Configure instructions for Copilot in repository Set up Copilot instructions Nov 27, 2025
Copilot AI requested a review from williambdean November 27, 2025 16:06
Copilot finished work on behalf of williambdean November 27, 2025 16:06
Comment on lines 120 to 127
- **PyMC** (>=5.24.1): Core probabilistic programming framework
- **PyTensor** (>=2.31.3): Tensor computation library
- **ArviZ** (>=0.13.0): Bayesian model diagnostics and visualization
- **pandas**: Data manipulation
- **NumPy**: Numerical computing
- **scikit-learn** (>=1.1.1): Machine learning utilities
- **xarray**: Multi-dimensional arrays with labels
- **Pydantic** (>=2.1.0): Data validation
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot dont specify versions. Just specific the file where to find these versions. Listing packages is fine

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed version numbers and added reference to pyproject.toml for version requirements (8172a67).

Comment on lines 148 to 150
from pymc_marketing.prior import Prior

prior = Prior("Normal", mu=0, sigma=1)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot This is stale. pymc_extras is used now

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to use from pymc_extras.prior import Prior (8172a67).

Co-authored-by: williambdean <57733339+williambdean@users.noreply.github.com>
Copilot AI requested a review from williambdean November 27, 2025 17:00
Copilot finished work on behalf of williambdean November 27, 2025 17:00
@williambdean
Copy link
Contributor

@copilot There are some incoming PR that use CLAUD . I want to make sure that there is no duplicated work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

✨ Set up Copilot instructions

2 participants