Skip to content

Relaxed model checks #154

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Relaxed model checks #154

wants to merge 2 commits into from

Conversation

rmkraus
Copy link
Collaborator

@rmkraus rmkraus commented Apr 17, 2025

During init of ChatNVIDIA, everything is now a warning, all of the errors have been removed.

The only exception is the a model must be found in order to continue. If the specified model cannot be found, the class will crash later, so it raises a ValueError during init.

@rmkraus rmkraus requested a review from raspawar April 17, 2025 20:35
@rmkraus rmkraus self-assigned this Apr 17, 2025
@rmkraus rmkraus added the enhancement New feature or request label Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant