use absolute imports

[asamal4]

Use absolute imports instead of relative for better clarity of the code structure & avoid any potential conflict with python packages.

Summary by CodeRabbit

• New Features

  • Expanded public API: more configuration objects, managers, pipeline helpers, and error types are now directly importable.
  • Simplified imports: users can import key components from stable, top-level package paths.

• Refactor

  • Standardized internal imports to absolute package paths for consistency and reliability.
  • Unified module namespaces without changing behavior or public signatures.

• Style

  • Harmonized exports across submodules for a clearer, more predictable developer experience.

[coderabbitai]

Walkthrough

This PR converts relative imports to absolute package imports across the codebase and updates several package init modules to re-export additional public symbols via all. No runtime logic or control flow changes are introduced.

Changes

Cohort / File(s)	Summary
Top-level package exports src/lightspeed_evaluation/__init__.py	Switch to absolute imports; significantly expands all to re-export configs, models, managers, pipeline, and exceptions.
Core package inits src/lightspeed_evaluation/core/__init__.py, .../core/api/__init__.py, .../core/llm/__init__.py, .../core/metrics/__init__.py, .../core/models/__init__.py, .../core/output/__init__.py, .../core/system/__init__.py	Convert relative to absolute imports. In core/llm/__init__.py, all expanded to include LLMConfig, LLMError, LLMManager, DeepEvalLLMManager, RagasLLMManager, validate_provider_env. Others keep exports unchanged.
Core script init src/lightspeed_evaluation/core/script/__init__.py	Absolute imports; all expanded to add ScriptExecutionError.
Core API client src/lightspeed_evaluation/core/api/client.py	Absolute imports for streaming parser, constants, models, exceptions; no logic changes.
Core embedding src/lightspeed_evaluation/core/embedding/manager.py, .../core/embedding/ragas.py	Absolute imports for configs, validators, and managers; behavior unchanged.
Core LLM src/lightspeed_evaluation/core/llm/manager.py	Absolute imports for models and env validator; no functional changes.
Core metrics src/lightspeed_evaluation/core/metrics/custom.py, .../metrics/deepeval.py, .../metrics/manager.py, .../metrics/ragas.py, .../metrics/script_eval.py	Absolute imports across metrics, LLM managers, models, scripts; no API or logic changes.
Core models src/lightspeed_evaluation/core/models/data.py, .../models/system.py	Constants imported via absolute path; no structural/model changes.
Core output src/lightspeed_evaluation/core/output/data_persistence.py, .../output/generator.py, .../output/statistics.py, .../output/visualization.py	Absolute imports for constants, models, statistics, visualization; signatures unchanged.
Core script manager src/lightspeed_evaluation/core/script/manager.py	Absolute import for ScriptExecutionError; logic unchanged.
Core system src/lightspeed_evaluation/core/system/env_validator.py, .../system/loader.py, .../system/setup.py, .../system/validator.py	Absolute imports for exceptions, models, setup; no functional changes.
Pipeline inits src/lightspeed_evaluation/pipeline/__init__.py, .../pipeline/evaluation/__init__.py	Absolute imports. pipeline/evaluation/__init__.py expands all to export APIDataAmender, ConversationProcessor, EvaluationErrorHandler, MetricsEvaluator.
Pipeline evaluation modules src/lightspeed_evaluation/pipeline/evaluation/amender.py, .../evaluation/errors.py, .../evaluation/evaluator.py, .../evaluation/pipeline.py, .../evaluation/processor.py	Absolute imports across core and pipeline components; behavior unchanged.
Runner src/lightspeed_evaluation/runner/__init__.py, .../runner/evaluation.py	Absolute imports for runner entry points and lazy imports; no changes to flow.

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• archive old eval and make lsc eval as primary #35 — Similar import path refactor and re-export adjustments across the package.
• Env setup/cleanup ability and verify through script #62 — Ties to script execution components; this PR exposes ScriptExecutionError alongside related wiring.
• API integration & refactoring #47 — Overlapping refactors of imports and all exports across core, pipeline, and models.

Suggested reviewers

• VladimirKadlec
• tisnik

Poem

A rabbit hops through import trails,
From dots to paths, it never fails.
Exports bloom like clover bright,
Names now shine in public light.
No code was bent, no flows askew—
Just clearer burrows, straight and true. 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title “use absolute imports” directly reflects the primary change of converting all relative imports to absolute paths, is concise and specific, and clearly conveys the main purpose of the pull request to reviewers.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e695669 and 910e51a.

📒 Files selected for processing (38)

• src/lightspeed_evaluation/__init__.py (2 hunks)
• src/lightspeed_evaluation/core/__init__.py (1 hunks)
• src/lightspeed_evaluation/core/api/__init__.py (1 hunks)
• src/lightspeed_evaluation/core/api/client.py (1 hunks)
• src/lightspeed_evaluation/core/embedding/manager.py (1 hunks)
• src/lightspeed_evaluation/core/embedding/ragas.py (1 hunks)
• src/lightspeed_evaluation/core/llm/__init__.py (1 hunks)
• src/lightspeed_evaluation/core/llm/manager.py (1 hunks)
• src/lightspeed_evaluation/core/metrics/__init__.py (1 hunks)
• src/lightspeed_evaluation/core/metrics/custom.py (1 hunks)
• src/lightspeed_evaluation/core/metrics/deepeval.py (1 hunks)
• src/lightspeed_evaluation/core/metrics/manager.py (1 hunks)
• src/lightspeed_evaluation/core/metrics/ragas.py (1 hunks)
• src/lightspeed_evaluation/core/metrics/script_eval.py (1 hunks)
• src/lightspeed_evaluation/core/models/__init__.py (1 hunks)
• src/lightspeed_evaluation/core/models/data.py (1 hunks)
• src/lightspeed_evaluation/core/models/system.py (1 hunks)
• src/lightspeed_evaluation/core/output/__init__.py (1 hunks)
• src/lightspeed_evaluation/core/output/data_persistence.py (1 hunks)
• src/lightspeed_evaluation/core/output/generator.py (1 hunks)
• src/lightspeed_evaluation/core/output/statistics.py (1 hunks)
• src/lightspeed_evaluation/core/output/visualization.py (1 hunks)
• src/lightspeed_evaluation/core/script/__init__.py (1 hunks)
• src/lightspeed_evaluation/core/script/manager.py (1 hunks)
• src/lightspeed_evaluation/core/system/__init__.py (2 hunks)
• src/lightspeed_evaluation/core/system/env_validator.py (1 hunks)
• src/lightspeed_evaluation/core/system/loader.py (2 hunks)
• src/lightspeed_evaluation/core/system/setup.py (1 hunks)
• src/lightspeed_evaluation/core/system/validator.py (1 hunks)
• src/lightspeed_evaluation/pipeline/__init__.py (1 hunks)
• src/lightspeed_evaluation/pipeline/evaluation/__init__.py (1 hunks)
• src/lightspeed_evaluation/pipeline/evaluation/amender.py (1 hunks)
• src/lightspeed_evaluation/pipeline/evaluation/errors.py (1 hunks)
• src/lightspeed_evaluation/pipeline/evaluation/evaluator.py (1 hunks)
• src/lightspeed_evaluation/pipeline/evaluation/pipeline.py (1 hunks)
• src/lightspeed_evaluation/pipeline/evaluation/processor.py (1 hunks)
• src/lightspeed_evaluation/runner/__init__.py (1 hunks)
• src/lightspeed_evaluation/runner/evaluation.py (2 hunks)

🧰 Additional context used

🧠 Learnings (2)

📚 Learning: 2025-09-18T23:59:37.026Z

Learnt from: asamal4
PR: lightspeed-core/lightspeed-evaluation#55
File: src/lightspeed_evaluation/core/system/validator.py:146-155
Timestamp: 2025-09-18T23:59:37.026Z
Learning: In the lightspeed-evaluation project, the DataValidator in `src/lightspeed_evaluation/core/system/validator.py` is intentionally designed to validate only explicitly provided user evaluation data, not resolved metrics that include system defaults. When turn_metrics is None, the system falls back to system config defaults, and this validation separation is by design.

Applied to files:

• src/lightspeed_evaluation/core/system/validator.py

📚 Learning: 2025-07-16T12:07:29.169Z

Learnt from: asamal4
PR: lightspeed-core/lightspeed-evaluation#19
File: lsc_agent_eval/tests/core/agent_goal_eval/test_script_runner.py:0-0
Timestamp: 2025-07-16T12:07:29.169Z
Learning: In the lsc_agent_eval package, the ScriptRunner class was modified to use absolute paths internally rather than documenting path normalization behavior, providing more predictable and consistent path handling.

Applied to files:

• src/lightspeed_evaluation/core/metrics/script_eval.py
• src/lightspeed_evaluation/core/script/manager.py

🧬 Code graph analysis (35)

src/lightspeed_evaluation/core/output/generator.py (3)

src/lightspeed_evaluation/core/models/data.py (1)

• EvaluationResult (185-224)

src/lightspeed_evaluation/core/output/statistics.py (2)

• calculate_basic_stats (9-35)
• calculate_detailed_stats (38-58)

src/lightspeed_evaluation/core/output/visualization.py (1)

• GraphGenerator (30-451)

src/lightspeed_evaluation/core/system/validator.py (2)

src/lightspeed_evaluation/core/models/data.py (1)

• EvaluationData (135-182)

src/lightspeed_evaluation/core/system/exceptions.py (1)

• DataValidationError (16-17)

src/lightspeed_evaluation/core/output/visualization.py (2)

src/lightspeed_evaluation/core/models/data.py (1)

• EvaluationResult (185-224)

src/lightspeed_evaluation/core/output/statistics.py (2)

• calculate_basic_stats (9-35)
• calculate_detailed_stats (38-58)

src/lightspeed_evaluation/core/output/statistics.py (1)

src/lightspeed_evaluation/core/models/data.py (1)

• EvaluationResult (185-224)

src/lightspeed_evaluation/core/metrics/ragas.py (5)

src/lightspeed_evaluation/core/embedding/manager.py (1)

• EmbeddingManager (11-37)

src/lightspeed_evaluation/core/embedding/ragas.py (1)

• RagasEmbeddingManager (10-31)

src/lightspeed_evaluation/core/llm/manager.py (1)

• LLMManager (10-117)

src/lightspeed_evaluation/core/llm/ragas.py (1)

• RagasLLMManager (84-111)

src/lightspeed_evaluation/core/models/data.py (1)

• TurnData (35-132)

src/lightspeed_evaluation/core/system/loader.py (1)

src/lightspeed_evaluation/core/system/setup.py (2)

• setup_environment_variables (10-28)
• setup_logging (31-93)

src/lightspeed_evaluation/core/output/__init__.py (3)

src/lightspeed_evaluation/core/output/data_persistence.py (1)

• save_evaluation_data (14-52)

src/lightspeed_evaluation/core/output/generator.py (1)

• OutputHandler (23-295)

src/lightspeed_evaluation/core/output/visualization.py (1)

• GraphGenerator (30-451)

src/lightspeed_evaluation/core/embedding/manager.py (2)

src/lightspeed_evaluation/core/models/system.py (2)

• EmbeddingConfig (67-95)
• SystemConfig (228-255)

src/lightspeed_evaluation/core/system/env_validator.py (1)

• validate_provider_env (74-95)

src/lightspeed_evaluation/core/metrics/deepeval.py (3)

src/lightspeed_evaluation/core/llm/deepeval.py (1)

• DeepEvalLLMManager (8-42)

src/lightspeed_evaluation/core/llm/manager.py (1)

• LLMManager (10-117)

src/lightspeed_evaluation/core/models/data.py (1)

• TurnData (35-132)

src/lightspeed_evaluation/core/script/__init__.py (2)

src/lightspeed_evaluation/core/script/manager.py (1)

• ScriptExecutionManager (14-117)

src/lightspeed_evaluation/core/system/exceptions.py (1)

• ScriptExecutionError (28-41)

src/lightspeed_evaluation/pipeline/__init__.py (1)

src/lightspeed_evaluation/pipeline/evaluation/pipeline.py (1)

• EvaluationPipeline (23-181)

src/lightspeed_evaluation/pipeline/evaluation/amender.py (3)

src/lightspeed_evaluation/core/api/client.py (1)

• APIClient (22-239)

src/lightspeed_evaluation/core/models/data.py (1)

• EvaluationData (135-182)

src/lightspeed_evaluation/core/system/exceptions.py (1)

• APIError (8-9)

src/lightspeed_evaluation/core/metrics/script_eval.py (2)

src/lightspeed_evaluation/core/models/data.py (1)

• EvaluationScope (227-238)

src/lightspeed_evaluation/core/script/manager.py (1)

• ScriptExecutionManager (14-117)

src/lightspeed_evaluation/runner/evaluation.py (5)

src/lightspeed_evaluation/core/system/loader.py (1)

• ConfigLoader (69-123)

src/lightspeed_evaluation/core/output/generator.py (1)

• OutputHandler (23-295)

src/lightspeed_evaluation/core/output/statistics.py (1)

• calculate_basic_stats (9-35)

src/lightspeed_evaluation/core/system/validator.py (1)

• DataValidator (71-307)

src/lightspeed_evaluation/pipeline/evaluation/pipeline.py (1)

• EvaluationPipeline (23-181)

src/lightspeed_evaluation/core/output/data_persistence.py (1)

src/lightspeed_evaluation/core/models/data.py (1)

• EvaluationData (135-182)

src/lightspeed_evaluation/core/system/setup.py (1)

src/lightspeed_evaluation/core/models/system.py (1)

• LoggingConfig (177-196)

src/lightspeed_evaluation/pipeline/evaluation/processor.py (7)

src/lightspeed_evaluation/core/metrics/manager.py (2)

• MetricLevel (10-14)
• MetricManager (17-136)

src/lightspeed_evaluation/core/models/data.py (4)

• EvaluationData (135-182)
• EvaluationRequest (241-294)
• EvaluationResult (185-224)
• TurnData (35-132)

src/lightspeed_evaluation/core/script/manager.py (1)

• ScriptExecutionManager (14-117)

src/lightspeed_evaluation/core/system/loader.py (1)

• ConfigLoader (69-123)

src/lightspeed_evaluation/pipeline/evaluation/amender.py (1)

• APIDataAmender (13-80)

src/lightspeed_evaluation/pipeline/evaluation/errors.py (1)

• EvaluationErrorHandler (10-85)

src/lightspeed_evaluation/pipeline/evaluation/evaluator.py (1)

• MetricsEvaluator (25-167)

src/lightspeed_evaluation/core/llm/manager.py (2)

src/lightspeed_evaluation/core/models/system.py (2)

• LLMConfig (33-64)
• SystemConfig (228-255)

src/lightspeed_evaluation/core/system/env_validator.py (1)

• validate_provider_env (74-95)

src/lightspeed_evaluation/runner/__init__.py (2)

src/lightspeed_evaluation/runner/evaluation.py (2)

• main (105-126)
• run_evaluation (12-102)

src/lightspeed_evaluation/pipeline/evaluation/pipeline.py (1)

• run_evaluation (117-155)

src/lightspeed_evaluation/core/metrics/manager.py (2)

src/lightspeed_evaluation/core/models/data.py (2)

• EvaluationData (135-182)
• TurnData (35-132)

src/lightspeed_evaluation/core/models/system.py (1)

• SystemConfig (228-255)

src/lightspeed_evaluation/core/metrics/__init__.py (3)

src/lightspeed_evaluation/core/metrics/custom.py (1)

• CustomMetrics (29-274)

src/lightspeed_evaluation/core/metrics/deepeval.py (1)

• DeepEvalMetrics (18-130)

src/lightspeed_evaluation/core/metrics/ragas.py (1)

• RagasMetrics (25-286)

src/lightspeed_evaluation/pipeline/evaluation/evaluator.py (10)

src/lightspeed_evaluation/core/embedding/manager.py (1)

• EmbeddingManager (11-37)

src/lightspeed_evaluation/core/llm/manager.py (1)

• LLMManager (10-117)

src/lightspeed_evaluation/core/metrics/custom.py (1)

• CustomMetrics (29-274)

src/lightspeed_evaluation/core/metrics/deepeval.py (1)

• DeepEvalMetrics (18-130)

src/lightspeed_evaluation/core/metrics/manager.py (2)

• MetricLevel (10-14)
• MetricManager (17-136)

src/lightspeed_evaluation/core/metrics/ragas.py (1)

• RagasMetrics (25-286)

src/lightspeed_evaluation/core/metrics/script_eval.py (1)

• ScriptEvalMetrics (16-55)

src/lightspeed_evaluation/core/models/data.py (2)

• EvaluationResult (185-224)
• EvaluationScope (227-238)

src/lightspeed_evaluation/core/script/manager.py (1)

• ScriptExecutionManager (14-117)

src/lightspeed_evaluation/core/system/loader.py (1)

• ConfigLoader (69-123)

src/lightspeed_evaluation/core/metrics/custom.py (3)

src/lightspeed_evaluation/core/llm/manager.py (1)

• LLMManager (10-117)

src/lightspeed_evaluation/core/metrics/tool_eval.py (1)

• evaluate_tool_calls (10-34)

src/lightspeed_evaluation/core/models/data.py (2)

• EvaluationScope (227-238)
• TurnData (35-132)

src/lightspeed_evaluation/pipeline/evaluation/__init__.py (5)

src/lightspeed_evaluation/pipeline/evaluation/amender.py (1)

• APIDataAmender (13-80)

src/lightspeed_evaluation/pipeline/evaluation/errors.py (1)

• EvaluationErrorHandler (10-85)

src/lightspeed_evaluation/pipeline/evaluation/evaluator.py (1)

• MetricsEvaluator (25-167)

src/lightspeed_evaluation/pipeline/evaluation/pipeline.py (1)

• EvaluationPipeline (23-181)

src/lightspeed_evaluation/pipeline/evaluation/processor.py (1)

• ConversationProcessor (37-228)

src/lightspeed_evaluation/core/models/__init__.py (2)

src/lightspeed_evaluation/core/models/api.py (3)

• APIRequest (30-77)
• APIResponse (80-116)
• AttachmentData (20-27)

src/lightspeed_evaluation/core/models/data.py (5)

• EvaluationData (135-182)
• EvaluationRequest (241-294)
• EvaluationResult (185-224)
• EvaluationScope (227-238)
• TurnData (35-132)

src/lightspeed_evaluation/pipeline/evaluation/errors.py (1)

src/lightspeed_evaluation/core/models/data.py (2)

• EvaluationData (135-182)
• EvaluationResult (185-224)

src/lightspeed_evaluation/core/__init__.py (5)

src/lightspeed_evaluation/core/llm/manager.py (1)

• LLMManager (10-117)

src/lightspeed_evaluation/core/models/data.py (3)

• EvaluationData (135-182)
• EvaluationResult (185-224)
• TurnData (35-132)

src/lightspeed_evaluation/core/models/system.py (2)

• LLMConfig (33-64)
• SystemConfig (228-255)

src/lightspeed_evaluation/core/system/loader.py (1)

• ConfigLoader (69-123)

src/lightspeed_evaluation/core/system/validator.py (1)

• DataValidator (71-307)

src/lightspeed_evaluation/core/system/env_validator.py (1)

src/lightspeed_evaluation/core/system/exceptions.py (1)

• LLMError (24-25)

src/lightspeed_evaluation/core/script/manager.py (1)

src/lightspeed_evaluation/core/system/exceptions.py (1)

• ScriptExecutionError (28-41)

src/lightspeed_evaluation/core/api/client.py (3)

src/lightspeed_evaluation/core/api/streaming_parser.py (1)

• parse_streaming_response (14-60)

src/lightspeed_evaluation/core/models/api.py (2)

• APIRequest (30-77)
• APIResponse (80-116)

src/lightspeed_evaluation/core/system/exceptions.py (1)

• APIError (8-9)

src/lightspeed_evaluation/core/embedding/ragas.py (1)

src/lightspeed_evaluation/core/embedding/manager.py (1)

• EmbeddingManager (11-37)

src/lightspeed_evaluation/pipeline/evaluation/pipeline.py (11)

src/lightspeed_evaluation/core/api/client.py (1)

• APIClient (22-239)

src/lightspeed_evaluation/core/metrics/manager.py (1)

• MetricManager (17-136)

src/lightspeed_evaluation/core/models/data.py (2)

• EvaluationData (135-182)
• EvaluationResult (185-224)

src/lightspeed_evaluation/core/output/data_persistence.py (1)

• save_evaluation_data (14-52)

src/lightspeed_evaluation/core/script/manager.py (1)

• ScriptExecutionManager (14-117)

src/lightspeed_evaluation/core/system/loader.py (1)

• ConfigLoader (69-123)

src/lightspeed_evaluation/core/system/validator.py (1)

• DataValidator (71-307)

src/lightspeed_evaluation/pipeline/evaluation/amender.py (1)

• APIDataAmender (13-80)

src/lightspeed_evaluation/pipeline/evaluation/errors.py (1)

• EvaluationErrorHandler (10-85)

src/lightspeed_evaluation/pipeline/evaluation/evaluator.py (1)

• MetricsEvaluator (25-167)

src/lightspeed_evaluation/pipeline/evaluation/processor.py (2)

• ConversationProcessor (37-228)
• ProcessorComponents (27-34)

src/lightspeed_evaluation/core/system/__init__.py (5)

src/lightspeed_evaluation/core/models/system.py (1)

• SystemConfig (228-255)

src/lightspeed_evaluation/core/system/exceptions.py (6)

• APIError (8-9)
• ConfigurationError (12-13)
• DataValidationError (16-17)
• EvaluationError (4-5)
• LLMError (24-25)
• MetricError (20-21)

src/lightspeed_evaluation/core/system/loader.py (1)

• ConfigLoader (69-123)

src/lightspeed_evaluation/core/system/setup.py (2)

• setup_environment_variables (10-28)
• setup_logging (31-93)

src/lightspeed_evaluation/core/system/validator.py (1)

• DataValidator (71-307)

src/lightspeed_evaluation/core/llm/__init__.py (6)

src/lightspeed_evaluation/core/llm/deepeval.py (1)

• DeepEvalLLMManager (8-42)

src/lightspeed_evaluation/core/llm/manager.py (1)

• LLMManager (10-117)

src/lightspeed_evaluation/core/llm/ragas.py (1)

• RagasLLMManager (84-111)

src/lightspeed_evaluation/core/models/system.py (1)

• LLMConfig (33-64)

src/lightspeed_evaluation/core/system/env_validator.py (1)

• validate_provider_env (74-95)

src/lightspeed_evaluation/core/system/exceptions.py (1)

• LLMError (24-25)

src/lightspeed_evaluation/core/api/__init__.py (1)

src/lightspeed_evaluation/core/api/client.py (1)

• APIClient (22-239)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: mypy
• GitHub Check: Pylinter
• GitHub Check: pydocstyle

🔇 Additional comments (32)

src/lightspeed_evaluation/core/system/env_validator.py (1)

5-5: LGTM! Import correctly converted to absolute path.

The conversion from relative to absolute import for LLMError is correct and aligns with the PR objective to improve code clarity.

src/lightspeed_evaluation/core/models/data.py (1)

9-9: LGTM! Absolute import correctly applied.

The import conversion maintains the same functionality while improving code clarity as intended by the PR.

src/lightspeed_evaluation/core/models/system.py (1)

7-30: LGTM! Absolute import path correctly applied for constants.

The conversion from relative (..constants) to absolute import properly updates the import path for all configuration constants without affecting functionality.

src/lightspeed_evaluation/core/output/visualization.py (1)

13-21: LGTM! All imports correctly converted to absolute paths.

The conversion of imports from relative to absolute paths is properly implemented for constants, models, and statistics modules, maintaining all existing functionality.

src/lightspeed_evaluation/pipeline/evaluation/amender.py (1)

6-8: LGTM! API and model imports correctly converted to absolute paths.

The conversion properly updates the import paths for APIClient, EvaluationData, and APIError to absolute imports, consistent with the PR objective.

src/lightspeed_evaluation/core/output/generator.py (1)

9-20: LGTM! All imports correctly updated to absolute paths.

The conversion of imports for constants, models, statistics, and visualization modules properly implements the absolute import pattern without affecting the output generation functionality.

src/lightspeed_evaluation/pipeline/__init__.py (1)

3-3: LGTM! Pipeline import correctly converted to absolute path.

The conversion of the EvaluationPipeline import to an absolute path is correct and maintains the public API exposure through __all__.

src/lightspeed_evaluation/core/embedding/ragas.py (1)

7-7: LGTM! Embedding manager import correctly converted to absolute path.

The conversion from relative to absolute import for EmbeddingManager is properly implemented, aligning with the PR's goal to improve code structure clarity.

src/lightspeed_evaluation/core/api/__init__.py (1)

3-3: LGTM! Import path correctly updated to absolute form.

The conversion from relative to absolute import is correct and aligns with the PR objective to improve code clarity.

src/lightspeed_evaluation/core/metrics/ragas.py (1)

17-21: LGTM! Import paths correctly updated to absolute form.

All five import statements have been properly converted from relative to absolute paths, maintaining the same functionality while improving code clarity.

src/lightspeed_evaluation/core/embedding/manager.py (1)

3-4: LGTM! Import paths correctly updated to absolute form.

The conversion from relative imports (..models, ..system.env_validator) to absolute package paths is correct and improves code clarity.

src/lightspeed_evaluation/core/system/validator.py (1)

10-15: LGTM! Import paths correctly updated to absolute form.

All import statements have been properly converted from relative to absolute paths, maintaining the same functionality while improving code clarity.

src/lightspeed_evaluation/pipeline/evaluation/processor.py (1)

7-21: LGTM! Import paths correctly updated to absolute form.

All import statements across multiple modules (core.metrics, core.models, core.script, core.system, pipeline.evaluation) have been properly converted from relative to absolute paths, maintaining the same functionality.

src/lightspeed_evaluation/pipeline/evaluation/errors.py (1)

5-5: LGTM! Import path correctly updated to absolute form.

The conversion from relative import (...core.models) to absolute path is correct and improves code clarity.

src/lightspeed_evaluation/core/metrics/manager.py (1)

6-7: LGTM! Import paths correctly updated to absolute form.

The conversion from relative imports to absolute paths with specific submodule references (.data, .system) is correct and improves code clarity.

src/lightspeed_evaluation/core/metrics/__init__.py (1)

3-5: LGTM! Import paths correctly updated to absolute form.

All three import statements have been properly converted from relative to absolute paths, maintaining the same public API while improving code clarity.

src/lightspeed_evaluation/core/script/manager.py (1)

9-9: LGTM! Import path correctly updated.

The absolute import path for ScriptExecutionError is correct and improves code clarity.

src/lightspeed_evaluation/core/system/setup.py (1)

7-7: LGTM! Import path correctly updated.

The absolute import for LoggingConfig is correct and aligns with the package structure.

src/lightspeed_evaluation/runner/__init__.py (1)

3-3: LGTM! Import path correctly updated.

The absolute import path maintains the public API while improving clarity.

src/lightspeed_evaluation/core/llm/manager.py (1)

6-7: LGTM! Import paths correctly updated.

Both absolute imports are correct and improve code maintainability.

src/lightspeed_evaluation/core/output/__init__.py (1)

3-5: LGTM! Import paths correctly updated.

All absolute imports are correct and the public API remains unchanged.

src/lightspeed_evaluation/runner/evaluation.py (2)

9-9: LGTM! Import path correctly updated.

The absolute import for ConfigLoader is correct.

---

38-41: LGTM! Import paths correctly updated.

All absolute imports for heavy modules are correct. The deferred import strategy (loading heavy modules after environment setup) is preserved.

src/lightspeed_evaluation/core/system/__init__.py (1)

4-38: LGTM! Import paths correctly updated.

All absolute imports are correct and follow a consistent pattern. The public API surface remains unchanged.

src/lightspeed_evaluation/__init__.py (1)

12-39: LGTM! Import paths correctly updated and public API expanded.

All absolute imports are correct. The expanded __all__ list makes more symbols available at the package level, which is a backward-compatible enhancement that improves the developer experience.

src/lightspeed_evaluation/pipeline/evaluation/pipeline.py (1)

6-18: LGTM! Absolute imports improve clarity.

The conversion from relative to absolute imports is correctly implemented. This change enhances code clarity by making import sources explicit and helps avoid potential conflicts with Python's module resolution, especially when using -m execution or in complex package structures.

src/lightspeed_evaluation/core/output/statistics.py (1)

6-6: LGTM!

The absolute import path is correct and consistent with the project-wide refactoring.

src/lightspeed_evaluation/core/metrics/deepeval.py (1)

13-15: LGTM!

The absolute import paths are correct and align with the codebase refactoring to improve import clarity.

src/lightspeed_evaluation/core/models/__init__.py (1)

3-23: LGTM! Package re-exports correctly updated.

The conversion to absolute imports within this package __init__.py is correct. All symbols declared in __all__ are properly imported from their respective submodules, maintaining the public API surface.

src/lightspeed_evaluation/core/llm/__init__.py (1)

3-17: LGTM! Public API appropriately expanded.

The absolute import paths are correct, and the __all__ expansion to include LLMConfig, LLMError, and validate_provider_env is consistent with the imported symbols. This extends the public API without breaking existing consumers.

src/lightspeed_evaluation/core/script/__init__.py (1)

3-9: LGTM! Export of ScriptExecutionError is appropriate.

The absolute imports are correct, and adding ScriptExecutionError to the public API makes sense for consumers who need to handle script execution failures.

src/lightspeed_evaluation/pipeline/evaluation/__init__.py (1)

3-15: LGTM! Significant public API expansion.

The absolute imports are correct, and the __all__ expansion exposes key evaluation pipeline components (APIDataAmender, ConversationProcessor, EvaluationErrorHandler, MetricsEvaluator) that may be useful for advanced users who want to customize or extend the evaluation flow. All exported symbols are properly imported.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[asamal4]

@VladimirKadlec @tisnik PTAL

[VladimirKadlec]

LGTM, thank you 👍

[tisnik]

LGTM