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

Add: Batch add to JS/TS bindings #173

Merged
merged 5 commits into from
Jul 31, 2023
Merged

Conversation

davvard
Copy link
Contributor

@davvard davvard commented Jul 31, 2023

No description provided.

@ashvardanian
Copy link
Contributor

It's better to use a lambda to invoke the insertion on every input entry rather than pushing them to std::vector-s. Let's refactor for efficiency.

@davvard
Copy link
Contributor Author

davvard commented Jul 31, 2023

@ashvardanian Yes you are right, it's already done.

@ashvardanian ashvardanian merged commit 3effb50 into unum-cloud:main-dev Jul 31, 2023
19 of 22 checks passed
@ashvardanian ashvardanian changed the title Add: Batch add Add: Batch add to JS/TS bindings Jul 31, 2023
@ashvardanian
Copy link
Contributor

🎉 This PR is included in version 0.23.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants