Skip to content

Add support for batch size parameter#16

Merged
Pringled merged 1 commit intoMinishLab:mainfrom
andreim14:patch-1
Jan 15, 2025
Merged

Add support for batch size parameter#16
Pringled merged 1 commit intoMinishLab:mainfrom
andreim14:patch-1

Conversation

@andreim14
Copy link
Contributor

This PR introduces a --batch-size parameter to the script, allowing users to control the number of texts processed in each batch during encoding.

Changes:

Benefits:

  • Improves processing speed and flexibility by leveraging user-defined batch sizes.
  • Helps users optimize memory usage based on their hardware setup.

Testing:

  • Verified that varying batch sizes work correctly without impacting the accuracy of embeddings.

Please review and let me know if further adjustments are needed!

Add batch size parameter to enhance encoding flexibility

- Introduced `--batch-size` argument to the parser to allow users to specify the batch size during encoding.
- Improved script flexibility for diverse hardware configurations by enabling user-defined batch sizes.
Copy link
Member

@Pringled Pringled left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @andreim14, thanks for creating this PR! LGTM, merging

@Pringled Pringled merged commit bed8a24 into MinishLab:main Jan 15, 2025
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