We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
There was an error while loading. Please reload this page.
1 parent 019a30e commit 76a2344Copy full SHA for 76a2344
1 file changed
refactoring-plan-182.md
@@ -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