Skip to content

fix: add source_url to taxonomy alignment output#407

Merged
Anket11 merged 4 commits into
mainfrom
development
Mar 31, 2026
Merged

fix: add source_url to taxonomy alignment output#407
Anket11 merged 4 commits into
mainfrom
development

Conversation

@Anket11
Copy link
Copy Markdown
Member

@Anket11 Anket11 commented Mar 31, 2026

What changed

  • Improved code readability by breaking long lines in function signatures, exception messages, and complex expressions.
  • Applied consistent line wrapping across data_access.py, llm_methods.py, llm_models/hugging_face_llm.py, llm_models/llama_cpp_handler.py, llm_models/llm_router.py, llm_models/model_loader.py, and services.py.

Why

  • To adhere to established code style guidelines (e.g., PEP 8 maximum line length) and enhance overall code readability and maintainability.

Notes

  • This PR introduces no functional changes; it is a purely stylistic refactor.

Anket11 and others added 4 commits March 30, 2026 22:02
@github-actions
Copy link
Copy Markdown

Code Hygiene

The following items are defined but not yet in active use. Routine cleanup — no impact on functionality.

  • laiser/llm_methods.py:87 — unused variable 'args' (100% confidence)
  • laiser/llm_models/llama_cpp_handler.py:31 — unused variable 'chat_format' (100% confidence)
  • laiser/llm_models/llama_cpp_handler.py:72 — unused variable 'exc' (100% confidence)
  • laiser/llm_models/llama_cpp_handler.py:72 — unused variable 'exc_type' (100% confidence)
  • laiser/llm_models/llama_cpp_handler.py:72 — unused variable 'tb' (100% confidence)
  • laiser/llm_models/llm_router.py:70 — unused variable 'args' (100% confidence)
  • laiser/llm_models/llm_router.py:85 — unused variable 'args' (100% confidence)
  • laiser/services.py:99 — unused variable 'raw_description' (100% confidence)

@github-actions
Copy link
Copy Markdown

Complexity Analysis

Average: C — Moderate (15.0)

Hotspots tracked below. These are functional, well-exercised components — listed for visibility as the codebase scales.

Grade Name Type Score
F FAISSIndexManager.initialize_index Method 51
E FAISSIndexManager.search_similar_skills Method 32
D ResponseParser._parse_skills_from_response Method 24
D FAISSIndexManager Class 22
C ResponseParser Class 17
C ResponseParser.parse_ksa_extraction_response Method 16
C ResponseParser.parse_skill_extraction_response Method 15
C LLMRouter._initialize_components Method 14
C openai_generate Function 14
C parse_output_vllm Function 13
C SkillExtractionService.extract_and_align_core Method 12
C SkillAlignmentService.align_skills_to_taxonomy Method 11
B load_model_from_vllm Function 10
B anthropic_generate Function 9
B ResponseParser.parse_ksa_details_response Method 8
B SkillAlignmentService Class 7
B get_completion_vllm Function 7
B SkillExtractionService Class 6
B fetch_model_output Function 6
B LlamaCppBackend.__init__ Method 6

@Anket11 Anket11 merged commit 50941c4 into main Mar 31, 2026
13 of 14 checks passed
@github-actions
Copy link
Copy Markdown

📦 Preview package published!

Install and test this PR:

pip install --index-url https://test.pypi.org/simple/ laiser-preview==0.4.0.1.dev20260331023515

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant