Skip to content

Conversation

@rvguha
Copy link
Contributor

@rvguha rvguha commented Dec 19, 2025

Summary

Restore Pi Labs LLM scoring provider that was accidentally deleted during provider consolidation in PR #14.

Changes

  • Moved Pi Labs provider from packages/bundles/models/nlweb_models/llm/pi_labs.py to proper location at packages/providers/pilabs/models/
  • Added complete package structure with pyproject.toml and README
  • Provider includes:
    • PiLabsProvider: LLM provider for Pi Labs scoring API
    • PiLabsClient: Async HTTP client with HTTP/2 support
    • Thread-safe client initialization
    • Scoring spec support for relevance evaluation

Configuration

To use, add to config.yaml:

llm:
  scoring:
    llm_type: pilabs
    endpoint: "http://localhost:8001/invocations"
    import_path: "nlweb_pilabs_models.llm"
    class_name: "PiLabsProvider"

Test plan

  • Package structure follows provider conventions
  • All imports are correct
  • README documents usage
  • pyproject.toml has proper dependencies

🤖 Generated with Claude Code

Restore Pi Labs LLM scoring provider that was accidentally deleted during
provider consolidation. Moved from bundles to proper location in providers.

New structure:
- packages/providers/pilabs/models/nlweb_pilabs_models/llm/pi_labs.py
- Includes PiLabsProvider and PiLabsClient
- Async scoring with httpx and HTTP/2 support
- Thread-safe client initialization
- pyproject.toml for package installation
- README with usage instructions

To use, configure in config.yaml:
  llm:
    scoring:
      llm_type: pilabs
      endpoint: "http://localhost:8001/invocations"
      import_path: "nlweb_pilabs_models.llm"
      class_name: "PiLabsProvider"

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@rvguha rvguha merged commit 51324ec into main Dec 19, 2025
2 checks passed
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