Skip to content

Dev #849

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

Draft
wants to merge 133 commits into
base: main
Choose a base branch
from
Draft

Dev #849

wants to merge 133 commits into from

Conversation

alashchev17
Copy link
Member

No description provided.

MarcMcIntosh and others added 30 commits May 28, 2025 19:08
- Update `current_model` usages to incorporate Option type for better error handling.
- Modify `ft_toolset` field to utilize Option type, reflecting potential absence of toolset data.
- Enhance safety by using `.ok_or` method to provide meaningful error messages when current_model is not set.
- Simplify certain sections by eliminating unnecessary variable initialization and comments.
- Optimize the connection initialization by directly passing required parameters.
JegernOUTT and others added 30 commits June 26, 2025 14:48
Eliminate the unused `resolve_chat_model` function from `caps.rs` and remove all references to the `tokenizer` in multiple files, simplifying function signatures and improving maintainability.
Replace the regex crate with fancy-regex to handle complex regex patterns more effectively. Update relevant imports and add unwrap_or(false) to is_match calls to safely handle regex errors. Modify Cargo.toml to include the fancy-regex dependency.
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.

5 participants