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.

Sign up for GitHub

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

Jump to bottom

Users/amitjoshi/gpt tokenizer integration #734

Merged
merged 29 commits into from
Oct 17, 2023
Merged

Users/amitjoshi/gpt tokenizer integration #734

merged 29 commits into from
Oct 17, 2023

Conversation

amitjoshi438
Copy link
Contributor

@amitjoshi438 amitjoshi438 commented Oct 9, 2023 •
edited
Loading

  • Added tokenizer package to calculate prompt tokens.
  • Changes are behind FCB: SELECTED_CODE_INFO_ENABLED

amitjoshi and others added 23 commits September 27, 2023 10:46
getting info about selected code
b0b83f0
showing label for selected lines of code
59cb3b5
add flag to disable feature
a48f1c8
updated responsiveness
8d96a0c
code formatting fix
321030b
@tyaginidhi
Merge branch 'main' into users/amitjoshi/labelForSelectedCode
9c3dcd7
removed log statement
49a6de4
Merge branch 'users/amitjoshi/labelForSelectedCode' of https://github…
fbc324b 
….com/microsoft/powerplatform-vscode into users/amitjoshi/labelForSelectedCode
handles empty selections
d9bef02
passing and showing user selected code to copilot
662cc05
setting fixed vertical height for user code
4fa0dad
Merge branch 'main' into users/amitjoshi/showingSelectedCodeInCopilot…
4317781 
…Chat
removed redundant code
fd28310
added comment
8b9ce0d
Merge branch 'main' into users/amitjoshi/showingSelectedCodeInCopilot…
7455aa9 
…Chat
added const for explainCode msg
2d78054
sending localized string to copilot webview
b8774a6
gpt-tokenizer integration
de8b869
user prompt with selected code and 'explain' check
07fc8d1
showing context cmd only when copilot registered
2b5efe2
update handling of empty snippet
810176c
Merge branch 'main' of https://github.com/microsoft/powerplatform-vscode
288b11c 
 into users/amitjoshi/gptTokenizerIntegration
updated when clause
51da821
@amitjoshi438 amitjoshi438 marked this pull request as ready for review October 17, 2023 04:03
@amitjoshi438 amitjoshi438 requested a review from a team as a code owner October 17, 2023 04:03
tyaginidhi
tyaginidhi approved these changes Oct 17, 2023
View reviewed changes
src/common/copilot/PowerPagesCopilot.ts Outdated Show resolved Hide resolved
@amitjoshi438 amitjoshi438 enabled auto-merge (squash) October 17, 2023 16:44
@amitjoshi438 amitjoshi438 merged commit 525a0d2 into main Oct 17, 2023
@amitjoshi438 amitjoshi438 deleted the users/amitjoshi/gptTokenizerIntegration branch October 17, 2023 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tyaginidhi tyaginidhi tyaginidhi approved these changes

@tehcrashxor tehcrashxor tehcrashxor approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@amitjoshi438 @tehcrashxor @tyaginidhi