Skip to content

Conversation

@tahmidul612
Copy link
Collaborator

No description provided.

@tahmidul612 tahmidul612 changed the title Fix/utils-modules fix: resolve import path errors causing module load failures Oct 21, 2025
@tahmidul612 tahmidul612 requested a review from Copilot October 21, 2025 15:28
Copy link

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 resolves import path errors by converting relative imports to absolute imports across the codebase, ensuring consistent module loading. Additionally, it applies code formatting improvements including trailing commas, string quote consistency, and line length adjustments.

  • Converts relative imports (..utils.exceptions) to absolute imports (insta_rag.utils.exceptions)
  • Removes unused imports (time in reranker.py, os in basic_usage.py)
  • Applies formatting fixes including trailing commas, string quotes, and multi-line formatting

Reviewed Changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/insta_rag/vectordb/qdrant.py Converts relative import to absolute for exceptions
src/insta_rag/retrieval/reranker.py Removes unused time import and applies formatting fixes (trailing commas, line length)
src/insta_rag/retrieval/query_generator.py Converts relative import to absolute for QueryGenerationError
src/insta_rag/retrieval/keyword_search.py Applies formatting fixes for multi-line statements and f-string consistency
src/insta_rag/models/response.py Formats multi-line field definition
src/insta_rag/models/document.py Formats ternary expression across multiple lines
src/insta_rag/embedding/openai.py Converts relative import to absolute for EmbeddingError
src/insta_rag/core/retrieval_method.py Removes unused List import, adds trailing commas, formats multi-line condition
src/insta_rag/core/config.py Converts relative import to absolute for ConfigurationError
src/insta_rag/core/client.py Converts relative imports to absolute for exceptions and pdf_processing
src/insta_rag/chunking/utils.py Converts single quotes to double quotes in regex patterns and formats function signature
src/insta_rag/chunking/semantic.py Converts relative import to absolute for ChunkingError
examples/basic_usage.py Removes unused os import and converts f-strings to regular strings where appropriate

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

@tahmidul612 tahmidul612 merged commit aeb650b into main Oct 21, 2025
1 check passed
@tahmidul612 tahmidul612 deleted the fix/utils-modules branch October 21, 2025 15:29
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