Code Quality Improvements: Resolve Ruff Linting Warnings

[RupeshMangalam21]

Changes

• Apply automatic formatting using ruff format

• Add comprehensive docstrings to all modules and functions

• Fix line length issues for better readability

• Improve code quality with proper exception handling and default arguments

  Fixes Code Quality Improvements: Address Ruff Linting Warnings #158

Testing

• Run test suite
• Verify no functionality changes
• Check error handling paths

Checklist

• Code follows project style guide
• Documentation has been updated
• All tests pass
• No new warnings from Ruff

Notes

This PR addresses the following Ruff warning categories:

• D100-D417: Docstring formatting
• E501: Line length
• B006, B008: Mutable defaults
• B904: Exception handling

[RupeshMangalam21]

Hey, @Sukh-P , while resolving the errors, i have come across this piece of code -

max_dropout and first_forecast_step have default values set to pd.Timedelta(0). Since pd.Timedelta creates a new object each time, this is technically safe in Python because each call gets a new instance. However, Ruff's B008 flags any function call in defaults, regardless of mutability. So to comply with the linter, we need to replace these defaults with None and set them inside the function.

Something like this - Changing max_dropout: pd.Timedelta = pd.Timedelta(0) to max_dropout: pd.Timedelta | None = None

would this be appropriate?