Skip to content

Commit 571ceac

Browse files
committed
docs: add refactoring plan for extracting complex methods (#182)
1 parent c7601f3 commit 571ceac

File tree

1 file changed

+18
-0
lines changed

1 file changed

+18
-0
lines changed

refactoring-plan-182.md

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
# Refactoring Plan for Issue #182: Extract Complex Methods
2+
3+
## Overview
4+
This PR addresses the complexity in `TimeSeriesModel.fit_arch` method by extracting it into smaller, focused methods.
5+
6+
## Changes to be implemented:
7+
1. Extract GARCH model creation into `_create_garch_model()`
8+
2. Extract EGARCH model creation into `_create_egarch_model()`
9+
3. Extract TGARCH model creation into `_create_tgarch_model()`
10+
4. Extract common parameter validation into `_validate_arch_params()`
11+
12+
## Files affected:
13+
- `src/tsbootstrap/time_series_model.py`
14+
15+
## Testing plan:
16+
- Ensure all existing tests pass
17+
- Add unit tests for each extracted method
18+
- Verify model creation behavior remains unchanged

0 commit comments

Comments
 (0)