[bug] RestrictToTopic validator occurs when disable_classifier is False and disable_llm is True

[railsstudent]

Describe the bug
When disable_classifier is False and disable_llm is True, the _inference_local method returns error on

candidate_topics = model_input["valid_topics"] + model_input["invalid_topics"]

The error message is

TypeError: unsupported operand type(s) for +: 'set' and 'set'

It is because

        valid_topics = set(metadata.get('valid_topics', self._valid_topics))
        invalid_topics = set(metadata.get('invalid_topics', self._invalid_topics))

return two sets and + operator cannot form a new array

To Reproduce
Steps to reproduce the behavior:

restrictToTopic = RestrictToTopic(
    valid_topics=["environment"],
    invalid_topics=["conspiracy", "food", "entertainment", "politics", "religion", "violence", "hate"],
    disable_classifier=False,
    disable_llm=True,
    on_fail=OnFailAction.EXCEPTION
)

result = restrictToTopic.validate(value="Large corporations should act responsibly socially and environmentally.  They should reduce carbon emission. One way to monitor their behavior is by reading their ESG Report.")
print(result.outcome)

Expected behavior
The _inference_local should return a found_topics array.

Library version:
Version 0.5.2

Additional context
Add any other context about the problem here.

[JosephCatrambone]

Good catch.

I think I might have fixed this incidentally as a side effect with this PR: tryolabs/restricttotopic#23

[railsstudent]

@JosephCatrambone Thank you. Do you know when guardrails will have a new release? I am evaluating guardrails and LLM guards. While guardrails ai has more broken validators and missing validators (e.g., prompt refusal), it has more validators for detecting hallucinations and better error messages. Both are very good for me.

[JosephCatrambone]

Hi @railsstudent -- the PR got approved and merged yesterday, though I don't know if it's in the private pypi repo yet.

EDIT: Linking PR so it shows up: tryolabs/restricttotopic#23

EDIT: Also a way old PR: tryolabs/restricttotopic#21

EDIT: Just tried. It's up on Pipy!

[JosephCatrambone]

I'm going to close this as resolved since the test in the first is passing, but if there's still a problem please feel free to reopen this ticket and I'll take another swing at it. Thanks again for reporting the issue.