Skip to content
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

docs: Fixed a small syntax issue in the ModelClass Update fine-tuning.md #1493

Merged
merged 1 commit into from
Dec 27, 2024

Conversation

mdqst
Copy link
Contributor

@mdqst mdqst commented Dec 27, 2024

I noticed a syntax issue in the ModelClass enum where a comma was missing after the EMBEDDING value. This would have caused a compilation error or unexpected behavior. I've added the necessary comma so the code functions as intended.

Here’s the fix I applied:

Before:

enum ModelClass {
    SMALL,    // Fast, efficient for simple tasks
    MEDIUM,   // Balanced performance and capability
    LARGE,    // Most capable but slower/more expensive
    EMBEDDING // Specialized for vector embeddings
    IMAGE     // Image generation capabilities
}

After:

enum ModelClass {
    SMALL,    // Fast, efficient for simple tasks
    MEDIUM,   // Balanced performance and capability
    LARGE,    // Most capable but slower/more expensive
    EMBEDDING, // Specialized for vector embeddings
    IMAGE     // Image generation capabilities
}

This minor adjustment ensures that the values are properly separated in the enum, preventing potential issues down the line.

@lalalune lalalune merged commit a930c84 into elizaOS:main Dec 27, 2024
3 of 6 checks passed
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.

2 participants