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

Support int8 embedding and uint8 embedding #1527

Merged
merged 32 commits into from
Jul 25, 2024

Conversation

yangzq50
Copy link
Contributor

@yangzq50 yangzq50 commented Jul 24, 2024

What problem does this PR solve?

Support int8 embedding and uint8 embedding
Fix issue #1516
Fix issue #1530

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Breaking Change (fix or feature that could cause existing functionality not to work as expected)
  • Refactoring
  • Test cases
  • Python SDK impacted, Need to update PyPI

@yangzq50 yangzq50 added ci PR can be test and removed ci PR can be test labels Jul 24, 2024
@yangzq50 yangzq50 force-pushed the int8_embedding branch 2 times, most recently from dc47e0f to aebb00d Compare July 25, 2024 01:49
@yangzq50 yangzq50 added ci PR can be test wip work in progress and removed ci PR can be test wip work in progress labels Jul 25, 2024
@yangzq50 yangzq50 force-pushed the int8_embedding branch 2 times, most recently from c7ea761 to 23cd421 Compare July 25, 2024 04:13
@yangzq50 yangzq50 removed the ci PR can be test label Jul 25, 2024
@yangzq50 yangzq50 added ci PR can be test and removed ci PR can be test labels Jul 25, 2024
@JinHai-CN JinHai-CN merged commit f7b9026 into infiniflow:main Jul 25, 2024
4 of 5 checks passed
@yangzq50 yangzq50 deleted the int8_embedding branch July 25, 2024 11:52
@yangzq50 yangzq50 restored the int8_embedding branch July 25, 2024 11:52
@yangzq50 yangzq50 deleted the int8_embedding branch July 25, 2024 12:11
@yingfeng yingfeng mentioned this pull request Aug 8, 2024
78 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci PR can be test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants