Skip to content

Add CLAUDE.md project documentation - #2

Closed
steveya wants to merge 8 commits into
mainfrom
claude/add-claude-documentation-4HfdF
Closed

steveya wants to merge 8 commits into
mainfrom
claude/add-claude-documentation-4HfdF

Conversation

@steveya

@steveya steveya commented Mar 20, 2026

Copy link
Copy Markdown
Owner

Summary

Added comprehensive project documentation file (CLAUDE.md) that serves as a reference guide for developers working on the volatility-forecast library.

Key Changes

  • New file: CLAUDE.md with complete project overview and developer guide
    • Project description and purpose
    • Detailed repository structure with all major modules and their responsibilities
    • Development environment setup instructions (Python 3.11/3.12, conda, pip)
    • Environment variable configuration (.env file)
    • Common commands for testing and running examples
    • CI/CD pipeline documentation (GitHub Actions)
    • Architecture patterns and model hierarchy explanation
    • Feature system design and key feature types
    • Data pipeline and evaluation methodology
    • Coding conventions and best practices
    • Configuration file references
    • Core and optional dependencies

Notable Details

  • Documents the scikit-learn compatible API design with BaseVolatilityModel
  • Explains the four main forecasting models (ES, STES, XGBoost-STES, VolGRU)
  • Clarifies point-in-time (PIT) alignment requirements for the data pipeline
  • Provides evaluation methodology including fixed-split training and Diebold-Mariano testing
  • Emphasizes compatibility with alphaforge conventions and minimal edit philosophy

https://claude.ai/code/session_013jwWg13tp461tdKsEV5h8u

Documents project structure, development workflows, architecture patterns,
key conventions, and commands for navigating the volatility-forecast codebase.

https://claude.ai/code/session_013jwWg13tp461tdKsEV5h8u
@steveya

steveya commented Mar 20, 2026

Copy link
Copy Markdown
Owner Author

@claude CI/CD failed. please fix

@steveya

steveya commented Mar 20, 2026

Copy link
Copy Markdown
Owner Author

@copilot CI/CD failed, please fix

Copilot AI commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

@steveya I've opened a new pull request, #3, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a new top-level CLAUDE.md developer reference guide for the volatility-forecast library, covering repository structure, development setup, CI, architecture, conventions, and dependencies.

Changes:

  • Introduces CLAUDE.md with a project overview and repository/module map.
  • Documents local dev workflow (Python versions, .env keys, common commands).
  • Summarizes core architecture (model hierarchy, features, PIT pipeline, evaluation) and config/dependency pointers.

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

Comment thread CLAUDE.md
Comment on lines +53 to +55
# Run examples (some require TIINGO_API)
python examples/demo_forecast.py
```

Copilot AI Mar 20, 2026

Copy link

Choose a reason for hiding this comment

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

The "Run examples" command implies pip install -e .[dev] is sufficient, but examples/demo_forecast.py imports alphaforge and will fail unless alphaforge is installed separately. Consider noting the additional install prerequisite (and any other optional deps needed by examples) right here to avoid setup confusion.

Copilot uses AI. Check for mistakes.
Comment thread CLAUDE.md Outdated
steveya and others added 2 commits March 19, 2026 23:35
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: steveya <7390489+steveya@users.noreply.github.com>
@steveya

steveya commented Mar 20, 2026

Copy link
Copy Markdown
Owner Author

@copilot CI/CD failed, please fix until pass

Copilot AI commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

@steveya I've opened a new pull request, #4, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 3 commits March 20, 2026 03:41
Co-authored-by: steveya <7390489+steveya@users.noreply.github.com>
Skip signature feature tests when esig is unavailable
Skip signature feature tests when esig is unavailable
@steveya steveya closed this Mar 20, 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.

4 participants