Skip to content

Conversation

@Pouyanpi
Copy link
Collaborator

Description

Removes the deprecated nemoguardrails.llm.params module that was scheduled for removal in v0.19.0. The llm_params context manager has been replaced by passing parameters directly to llm_call() using
the llm_params argument.

Modified Files

  • nemoguardrails/llm/params.py (deleted - 131 lines removed)
  • tests/test_llm_params.py (deleted - 321 lines removed)

References

@Pouyanpi Pouyanpi added this to the v0.19.0 milestone Oct 27, 2025
@Pouyanpi Pouyanpi self-assigned this Oct 27, 2025
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Greptile Overview

Greptile Summary

This PR removes the deprecated nemoguardrails.llm.params module and its associated tests as part of a planned deprecation cycle. The module provided a context manager pattern (llm_params()) for temporarily adjusting LLM parameters, which was deprecated in v0.17.0 and scheduled for removal in v0.19.0. The functionality has been replaced by passing parameters directly to llm_call() using the llm_params argument. This cleanup simplifies the API surface by removing 131 lines of production code and 321 lines of test code. The change aligns with the project's architectural shift toward leveraging LangChain's universal .bind() method for parameter binding across all model packages, reducing complexity in the NeMo Guardrails LLM integration layer.

Important Files Changed

Filename Score Overview
nemoguardrails/llm/params.py 5/5 Removed deprecated module containing LLMParams context manager class and helper functions (131 lines)
tests/test_llm_params.py 5/5 Deleted comprehensive test suite for the removed params module (321 lines)

Confidence score: 5/5

  • This PR is safe to merge with minimal risk
  • The removal follows a documented deprecation schedule (deprecated v0.17.0, removal planned for v0.19.0), and the functionality has been replaced by a simpler direct parameter-passing approach that has been available for multiple versions
  • No files require special attention; this is straightforward cleanup of deprecated code with no remaining references

Sequence Diagram

sequenceDiagram
    participant Developer
Loading

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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.

3 participants