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

Download from hub #1011

Closed
wants to merge 12 commits into from
Closed

Conversation

bikash119
Copy link

@bikash119 bikash119 commented Oct 2, 2024

This PR is not needed anymore.

bikash119 and others added 12 commits September 24, 2024 13:58
…beddings should be normalized

- Added testcases to test normalize embeddings
Accept recommended suggestion

Co-authored-by: David Berenstein <david.m.berenstein@gmail.com>
- Incorporated changes suggested in review comments.
- use atexit to forcefully invoke cleanup
- Add test_encode_batch_consistency to ensure consistent results
- Test large batch processing capability
- Verify embedding dimensions and count for different batch sizes
@bikash119 bikash119 marked this pull request as ready for review October 2, 2024 12:09
@bikash119 bikash119 marked this pull request as draft October 2, 2024 12:11
@bikash119 bikash119 closed this Oct 2, 2024
@bikash119 bikash119 deleted the download_from_hub branch October 2, 2024 12:13
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.

1 participant