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

feat!: get completions camel case #48

Merged
merged 3 commits into from
Nov 15, 2023
Merged

Conversation

McPatate
Copy link
Member

@McPatate McPatate commented Nov 15, 2023

  • make API camelCase for consistency
  • sync docs with incremental changes
  • make sure indexes are not out of bounds and avoid panicking using safe methods to get value from ropes

Should fix huggingface/llm-vscode#107

@McPatate McPatate force-pushed the feat!/getComletions_camelCase branch from ba46118 to b462f09 Compare November 15, 2023 11:44
@McPatate McPatate changed the title Feat!/get comletions camel case feat!: get completions camel case Nov 15, 2023
Copy link

Repository name Source type Average hole completion time (s) Pass percentage
lancedb/lance github 90.3676 69.465034%
helix-editor/helix github 77.732796 70%
huggingface/huggingface_hub github 18.878199 100%
encode/starlette github 36.6834 89.73298%
tkaitchuck/constrandom github 2.958 30.000002%
mmaitre314/picklescan github 4.363 100%
tiangolo/fastapi github 88.2744 79.91882%
colinhacks/zod github 53.7531 80%
smol-rs/async-executor github 8.4392 50%
lancedb/lancedb github 70.88979 80%
jaemk/cached github 33.6503 100%
simple local 2.81704 93%
gcanti/io-ts github 61.9263 100%
Total -- 26.18575 85.414406%

Note: The "hole completion time" represents the full process of:

  • replacing the code from the file with a completion from the model
  • building the project
  • running the tests

@McPatate McPatate merged commit 3ad64a3 into main Nov 15, 2023
1 check passed
@McPatate McPatate deleted the feat!/getComletions_camelCase branch November 15, 2023 12:36
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.

Extension panicking
1 participant