Skip to content

Conversation

miyoungc
Copy link
Collaborator

Description

As of version 0.14.1, an extra backslash is required at the end of NIMs model endpoints.

Related Issue(s)

Checklist

  • I've read the CONTRIBUTING guidelines.
  • I've updated the documentation if applicable.
  • I've added tests if applicable.
  • @mentions of the person or team responsible for reviewing proposed changes.

@miyoungc miyoungc added this to the v0.16.0 milestone Aug 21, 2025
@miyoungc miyoungc self-assigned this Aug 21, 2025
@miyoungc miyoungc added the documentation Improvements or additions to documentation label Aug 21, 2025
Copy link
Contributor

Documentation preview

https://nvidia.github.io/NeMo-Guardrails/review/pr-1345

@codecov-commenter
Copy link

codecov-commenter commented Aug 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.59%. Comparing base (a5e8b4c) to head (92bc2bd).
⚠️ Report is 4 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1345      +/-   ##
===========================================
+ Coverage    71.04%   71.59%   +0.54%     
===========================================
  Files          162      168       +6     
  Lines        16398    16862     +464     
===========================================
+ Hits         11650    12072     +422     
- Misses        4748     4790      +42     
Flag Coverage Δ
python 71.59% <ø> (+0.54%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 16 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Pouyanpi
Copy link
Collaborator

thank you @miyoungc for bringing this to my attention 👍🏻 I think this should be only happening to jailbreak detection and not the others. I have opened a PR that fixes the bug #1346.

If we could get that PR merged and include it in 0.16.0 we can close this docs update.

Copy link
Collaborator

@Pouyanpi Pouyanpi left a comment

Choose a reason for hiding this comment

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

Let's see if we need to merge this PR or the bug fix.

Copy link
Collaborator

@Pouyanpi Pouyanpi left a comment

Choose a reason for hiding this comment

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

Thank you @miyoungc 👍🏻

@Pouyanpi Pouyanpi merged commit baca9ec into NVIDIA-NeMo:develop Aug 26, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants