Skip to content

Refactor: separate estimators into separate files#67

Merged
TomeHirata merged 3 commits into
mainfrom
refactor/separate-files
Sep 10, 2025
Merged

Refactor: separate estimators into separate files#67
TomeHirata merged 3 commits into
mainfrom
refactor/separate-files

Conversation

@TomeHirata
Copy link
Copy Markdown
Collaborator

Refactor the init.py and separate the contents into multiple files

@TomeHirata TomeHirata requested a review from Copilot September 10, 2025 02:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the __init__.py file by extracting classes into separate modules to improve code organization and maintainability.

  • Moves base class and concrete estimator implementations to dedicated modules
  • Creates separate files for stratified, simple, and local distribution estimators
  • Simplifies the main __init__.py to focus on imports and exports

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
dte_adj/__init__.py Refactored to import classes from separate modules instead of defining them inline
dte_adj/base.py Contains the base DistributionEstimatorBase abstract class with common functionality
dte_adj/stratified.py Contains stratified distribution estimator implementations
dte_adj/simple.py Contains simple distribution estimator implementations that inherit from stratified classes
dte_adj/local.py Contains local distribution estimator implementations for LDTE and LPTE

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread dte_adj/stratified.py Outdated
Comment thread dte_adj/stratified.py Outdated
Comment thread dte_adj/stratified.py Outdated
Comment thread dte_adj/stratified.py Outdated
Comment thread dte_adj/stratified.py Outdated
@TomeHirata TomeHirata merged commit e4fb68d into main Sep 10, 2025
6 checks passed
@TomeHirata TomeHirata deleted the refactor/separate-files branch September 10, 2025 02:30
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