Skip to content

✨ feat(app): add MD5 verification for FastText model integrity #9

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

Merged
merged 5 commits into from
Jan 11, 2025

Conversation

The `detect_langs` function was removed to streamline the codebase.
This change eliminates redundant functionality and encourages the use
of the updated `detect_language` method, enhancing maintainability.
- Adjusted `retry_max` to 2, `sleep_max` to 5s, `timeout` to 7s
- Changed logging from error to warning for load failures
- Defaulted `low_memory` to False for better performance

These changes improve resilience and performance in model handling.
Introduce MD5 hash verification for the FastText model download.
This ensures the integrity of the model file, reducing prediction
errors due to corrupted downloads.
- Added guidelines for handling line breaks in input text.
- Clarified parameter descriptions for better user understanding.
- Highlighted the importance of large models for accuracy.
@sudoskys sudoskys self-assigned this Jan 11, 2025
@sudoskys sudoskys added the enhancement New feature or request label Jan 11, 2025
🔄 Updated the project version to 0.2.4 for new changes.
@sudoskys sudoskys merged commit 364d4f5 into main Jan 11, 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