Arima

[rdhakan13]

No description provided.

[Copilot]

Pull Request Overview

This PR, titled "Arima", primarily focuses on adding type annotations and improving code formatting and logging consistency across multiple modules in the project. Key changes include:

• Addition of explicit return types and parameter type hints throughout the codebase.
• Refactoring of logging statements and code formatting for improved readability.
• Updates in configuration files and workflow triggers to align with the current project standards.

Reviewed Changes

Copilot reviewed 24 out of 24 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/preprocessing/data_loader.py	Added type annotations and reformatted method bodies.
src/preprocessing/data_cleaner.py	Introduced return type hints and minor refactoring.
src/postprocessing/mlflowtracker.py	Added a new MLflow tracker implementation.
src/models/base.py and 2DCNN-GRU.py	Enhanced type hints and improved constructor formatting.
src/data/*.py	Enforced mandatory parameters and updated type annotations.
src/common/*.py	Removed decorators and refined utility functions with type hints.
scripts/pipeline/*.py	Removed redundant whitespace for a cleaner pipeline execution.
pyproject.toml and .github/workflows/unit-tests.yml	Updated dependencies, mypy configurations, and CI triggers.

Comments suppressed due to low confidence (2)

src/data/google_news.py:30

• The error message 'Root self.raw_dir must be a non-empty string' is confusing. Consider revising it to 'Root directory must be a non-empty string'.

raise ValueError("Root self.raw_dir must be a non-empty string")

src/data/etherscan.py:28

• Duplicate assignment to 'self.ticker' detected; remove the redundant assignment to prevent confusion.

self.ticker = ticker