mirror of https://github.com/jpawlowski/hass.tibber_prices.git synced 2026-03-30 05:13:40 +00:00

Julian Pawlowski 457fa7c03f refactor(periods): merge adjacent periods and remove is_extension logic

BREAKING CHANGE: Period overlap resolution now merges adjacent/overlapping periods
instead of marking them as extensions. This simplifies automation logic and provides
clearer period boundaries for users.

Previous Behavior:
- Adjacent periods created by relaxation were marked with is_extension=true
- Multiple short periods instead of one continuous period
- Complex logic needed to determine actual period length in automations

New Behavior:
- Adjacent/overlapping periods are merged into single continuous periods
- Newer period's relaxation attributes override older period's
- Simpler automation: one period = one continuous time window

Changes:
- Period Overlap Resolution (new file: period_overlap.py):
  * Added merge_adjacent_periods() to combine periods and preserve attributes
  * Rewrote resolve_period_overlaps() with simplified merge logic
  * Removed split_period_by_overlaps() (no longer needed)
  * Removed is_extension marking logic
  * Removed unused parameters: min_period_length, baseline_periods

- Relaxation Strategy (relaxation.py):
  * Removed all is_extension filtering from period counting
  * Simplified standalone counting to just len(periods)
  * Changed from period_merging import to period_overlap import
  * Added MAX_FLEX_HARD_LIMIT constant (0.50)
  * Improved debug logging for merged periods

- Code Quality:
  * Fixed all remaining linter errors (N806, PLR2004, PLR0912)
  * Extracted magic values to module-level constants:
    - FLEX_SCALING_THRESHOLD = 0.20
    - SCALE_FACTOR_WARNING_THRESHOLD = 0.8
    - MAX_FLEX_HARD_LIMIT = 0.50
  * Added appropriate noqa comments for unavoidable patterns

- Configuration (from previous work in this session):
  * Removed CONF_RELAXATION_STEP_BEST, CONF_RELAXATION_STEP_PEAK
  * Hard-coded 3% relaxation increment for reliability
  * Optimized defaults: RELAXATION_ATTEMPTS 8→11, ENABLE_MIN_PERIODS False→True,
    MIN_PERIODS undefined→2
  * Removed relaxation_step UI fields from config flow
  * Updated all 5 translation files

- Documentation:
  * Updated period_handlers/__init__.py: period_merging → period_overlap
  * No user-facing docs changes needed (already described continuous periods)

Rationale - Period Merging:
User experience was complicated by fragmented periods:
- Automations had to check multiple adjacent periods
- Binary sensors showed ON/OFF transitions within same cheap time
- No clear way to determine actual continuous period length

With merging:
- One continuous cheap time = one period
- Binary sensor clearly ON during entire period
- Attributes show merge history via merged_from dict
- Relaxation info preserved from newest/highest flex period

Rationale - Hard-Coded Relaxation Increment:
The configurable relaxation_step parameter proved problematic:
- High base flex + high step → rapid explosion (40% base + 10% step → 100% in 6 steps)
- Users don't understand the multiplicative nature
- 3% increment provides optimal balance: 11 attempts to reach 50% hard cap

Impact:
- Existing installations: Periods may appear longer (merged instead of split)
- Automations benefit from simpler logic (no is_extension checks needed)
- Custom relaxation_step values will use new 3% increment
- Users may need to adjust relaxation_attempts if they relied on high step sizes

2025-11-19 20:16:58 +00:00

6.9 KiB

Raw Blame History

Developer Documentation

This section contains documentation for contributors and maintainers of the Tibber Prices integration.

📚 Developer Guides

Setup - DevContainer, environment setup, and dependencies
Architecture - Code structure, patterns, and conventions
Period Calculation Theory - Mathematical foundations, Flex/Distance interaction, Relaxation strategy
Timer Architecture - Timer system, scheduling, coordination (3 independent timers)
Caching Strategy - Cache layers, invalidation, debugging
Testing - How to run tests and write new test cases
Release Management - Release workflow and versioning process
Coding Guidelines - Style guide, linting, and best practices
Refactoring Guide - How to plan and execute major refactorings

🤖 AI Documentation

The main AI/Copilot documentation is in AGENTS.md. This file serves as long-term memory for AI assistants and contains:

Detailed architectural patterns
Code quality rules and conventions
Development workflow guidance
Common pitfalls and anti-patterns
Project-specific patterns and utilities

Important: When proposing changes to patterns or conventions, always update AGENTS.md to keep AI guidance consistent.

AI-Assisted Development

This integration is developed with extensive AI assistance (GitHub Copilot, Claude, and other AI tools). The AI handles:

Pattern Recognition: Understanding and applying Home Assistant best practices
Code Generation: Implementing features with proper type hints, error handling, and documentation
Refactoring: Maintaining consistency across the codebase during structural changes
Translation Management: Keeping 5 language files synchronized
Documentation: Generating and maintaining comprehensive documentation

Quality Assurance:

Automated linting with Ruff (120-char line length, max complexity 25)
Home Assistant's type checking and validation
Real-world testing in development environment
Code review by maintainer before merging

Benefits:

Rapid feature development while maintaining quality
Consistent code patterns across all modules
Comprehensive documentation maintained alongside code
Quick bug fixes with proper understanding of context

Limitations:

AI may occasionally miss edge cases or subtle bugs
Some complex Home Assistant patterns may need human review
Translation quality depends on AI's understanding of target language
User feedback is crucial for discovering real-world issues

If you're working with AI tools on this project, the AGENTS.md file provides the context and patterns that ensure consistency.

🚀 Quick Start for Contributors

Fork and clone the repository
Open in DevContainer (VS Code: "Reopen in Container")
Run setup: ./scripts/setup (happens automatically via postCreateCommand)
Start development environment: ./scripts/develop
Make your changes following the Coding Guidelines
Run linting: ./scripts/lint
Validate integration: ./scripts/hassfest
Test your changes in the running Home Assistant instance
Commit using Conventional Commits format
Open a Pull Request with clear description

🛠️ Development Tools

The project includes several helper scripts in ./scripts/:

bootstrap - Initial setup of dependencies
develop - Start Home Assistant in debug mode (auto-cleans .egg-info)
clean - Remove build artifacts and caches
lint - Auto-fix code issues with ruff
lint-check - Check code without modifications (CI mode)
hassfest - Validate integration structure (JSON, Python syntax, required files)
setup - Install development tools (git-cliff, @github/copilot)
prepare-release - Prepare a new release (bump version, create tag)
generate-release-notes - Generate release notes from commits

📦 Project Structure

custom_components/tibber_prices/
├── __init__.py           # Integration setup
├── coordinator.py        # Data update coordinator with caching
├── api.py               # Tibber GraphQL API client
├── price_utils.py       # Price enrichment functions
├── average_utils.py     # Average calculation utilities
├── sensor/              # Sensor platform (package)
│   ├── __init__.py      #   Platform setup
│   ├── core.py          #   TibberPricesSensor class
│   ├── definitions.py   #   Entity descriptions
│   ├── helpers.py       #   Pure helper functions
│   └── attributes.py    #   Attribute builders
├── binary_sensor.py     # Binary sensor platform
├── entity_utils/        # Shared entity helpers
│   ├── icons.py         #   Icon mapping logic
│   ├── colors.py        #   Color mapping logic
│   └── attributes.py    #   Common attribute builders
├── services.py          # Custom services
├── config_flow.py       # UI configuration flow
├── const.py             # Constants and helpers
├── translations/        # Standard HA translations
└── custom_translations/ # Extended translations (descriptions)

🔍 Key Concepts

DataUpdateCoordinator Pattern:

Centralized data fetching and caching
Automatic entity updates on data changes
Persistent storage via Store
Quarter-hour boundary refresh scheduling

Price Data Enrichment:

Raw API data is enriched with statistical analysis
Trailing/leading 24h averages calculated per interval
Price differences and ratings added
All via pure functions in price_utils.py

Translation System:

Dual system: /translations/ (HA schema) + /custom_translations/ (extended)
Both must stay in sync across all languages (de, en, nb, nl, sv)
Async loading at integration setup

🧪 Testing

# Validate integration structure
./scripts/hassfest

# Run all tests
pytest tests/

# Run specific test file
pytest tests/test_coordinator.py

# Run with coverage
pytest --cov=custom_components.tibber_prices tests/

📝 Documentation Standards

User-facing docs go in docs/user/
Developer docs go in docs/development/
AI guidance goes in AGENTS.md
Use clear examples and code snippets
Keep docs up-to-date with code changes

🤝 Contributing

See CONTRIBUTING.md for detailed contribution guidelines, code of conduct, and pull request process.

📄 License

This project is licensed under LICENSE.

Note: This documentation is for developers. End users should refer to the User Documentation.

6.9 KiB Raw Blame History