Skip to content

docs(guardrails): fix typos, model references, and links in guardrail catalog#1651

Merged
Pouyanpi merged 1 commit intodevelopfrom
docs/configure-rails/guardrail-catalog.md
Feb 23, 2026
Merged

docs(guardrails): fix typos, model references, and links in guardrail catalog#1651
Pouyanpi merged 1 commit intodevelopfrom
docs/configure-rails/guardrail-catalog.md

Conversation

@Pouyanpi
Copy link
Collaborator

Description

  • Fix gpt-3.5-turbo-instruct to gpt-4o-mini
  • Fix typos: idenfier to identifier, tourch to torch
  • Fix Patronus AI URL missing https://

… catalog

- Fix gpt-3.5-turbo-instruct to gpt-4o-mini
- Fix typos: idenfier to identifier, tourch to torch
- Fix Patronus AI URL missing https://
@Pouyanpi Pouyanpi added this to the v0.21 milestone Feb 18, 2026
@Pouyanpi Pouyanpi requested a review from miyoungc February 18, 2026 13:32
@Pouyanpi Pouyanpi self-assigned this Feb 18, 2026
@Pouyanpi Pouyanpi added the documentation Improvements or additions to documentation label Feb 18, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 18, 2026

Greptile Summary

Documentation-only PR that fixes three items in docs/configure-rails/guardrail-catalog.md:

  • Updates the example model from gpt-3.5-turbo-instruct to gpt-4o-mini, consistent with other content safety example configs in the repository
  • Fixes typo "idenfier" → "identifier" (line 474)
  • Fixes typo "tourch" → "torch" (line 873)

Issue: The PR description mentions fixing the Patronus AI URL to include https://, but this change is not included in the actual diff. The URL on line 1073 still reads (www.patronus.ai) without the protocol prefix, which will cause the link to be non-functional in most Markdown renderers.

Confidence Score: 4/5

  • Documentation-only changes with no code impact; safe to merge after addressing the missing URL fix.
  • All three included changes are correct and improve documentation quality. The only issue is the Patronus AI URL fix described in the PR but not present in the diff, which should be added before merging.
  • docs/configure-rails/guardrail-catalog.md — the Patronus AI URL fix mentioned in the PR description is missing from the changeset.

Important Files Changed

Filename Overview
docs/configure-rails/guardrail-catalog.md Three documentation fixes (model name, two typos) are correct, but the Patronus AI URL fix mentioned in the PR description is missing from the actual changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["guardrail-catalog.md"] --> B["Fix 1: Model Reference\ngpt-3.5-turbo-instruct → gpt-4o-mini\n(Line 458)"]
    A --> C["Fix 2: Typo\nidenfier → identifier\n(Line 474)"]
    A --> D["Fix 3: Typo\ntourch → torch\n(Line 873)"]
    A --> E["Missing Fix: Patronus URL\nwww.patronus.ai → https://www.patronus.ai\n(Line 1073)"]
    style E stroke:#ff6666,stroke-width:2px
Loading

Last reviewed commit: cadcb98

@github-actions
Copy link
Contributor

Documentation preview

https://nvidia-nemo.github.io/Guardrails/review/pr-1651

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.

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 18, 2026

Additional Comments (1)

docs/configure-rails/guardrail-catalog.md
Missing Patronus AI URL fix

The PR description mentions fixing the Patronus AI URL to include https://, but this change is not present in the diff. Line 1073 still has [Patronus AI](www.patronus.ai) — the link will not work correctly in most Markdown renderers without the https:// protocol prefix.

The NeMo Guardrails library supports hallucination detection in RAG systems using [Patronus AI](https://www.patronus.ai)'s Lynx model. The model is hosted on Hugging Face and comes in both a 70B parameters (see [here](https://huggingface.co/PatronusAI/Patronus-Lynx-70B-Instruct)) and 8B parameters (see [here](https://huggingface.co/PatronusAI/Patronus-Lynx-8B-Instruct)) variant.
Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/configure-rails/guardrail-catalog.md
Line: 1073

Comment:
**Missing Patronus AI URL fix**

The PR description mentions fixing the Patronus AI URL to include `https://`, but this change is not present in the diff. Line 1073 still has `[Patronus AI](www.patronus.ai)` — the link will not work correctly in most Markdown renderers without the `https://` protocol prefix.

```suggestion
The NeMo Guardrails library supports hallucination detection in RAG systems using [Patronus AI](https://www.patronus.ai)'s Lynx model. The model is hosted on Hugging Face and comes in both a 70B parameters (see [here](https://huggingface.co/PatronusAI/Patronus-Lynx-70B-Instruct)) and 8B parameters (see [here](https://huggingface.co/PatronusAI/Patronus-Lynx-8B-Instruct)) variant.
```

How can I resolve this? If you propose a fix, please make it concise.

@Pouyanpi Pouyanpi merged commit a55f2c0 into develop Feb 23, 2026
4 checks passed
@Pouyanpi Pouyanpi deleted the docs/configure-rails/guardrail-catalog.md branch February 23, 2026 10:07
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.

2 participants